UPDATE: OFBiz code linting issues

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

UPDATE: OFBiz code linting issues

Suraj Khurana-2
Hello team,

Sometimes back, we introduced sonar and checkstyle linting tools integrated
with OFBiz.
We as a community are serious not only about features being introduced in
OFBiz but also code quality being added in the repo.
So we started correcting issues identified by the linting tool (checkstyle)
and there were nearly ~*40,000* of those in the whole codebase.

Slowly we keep on discussing it on different mail threads and move towards
conclusions and fix those into the code base.
We also incorporated a pre-push commit hook to ensure that no new issues
are being introduced by committers.

I am glad to announce that we have fixed around *99%* of these issues
successfully.
Kudos to everyone involved in this effort !!

--
Best Regards,
Suraj Khurana
Senior Technical Consultant
Reply | Threaded
Open this post in threaded view
|

Re: UPDATE: OFBiz code linting issues

Jacques Le Roux
Administrator
Thanks Suraj, and to all who worked on it.

I know it's not yet finished, but I think the next similar tasks would be to look at the FIXMEs and later TODOs...

Jacques

Le 26/09/2020 à 12:36, Suraj Khurana a écrit :

> Hello team,
>
> Sometimes back, we introduced sonar and checkstyle linting tools integrated
> with OFBiz.
> We as a community are serious not only about features being introduced in
> OFBiz but also code quality being added in the repo.
> So we started correcting issues identified by the linting tool (checkstyle)
> and there were nearly ~*40,000* of those in the whole codebase.
>
> Slowly we keep on discussing it on different mail threads and move towards
> conclusions and fix those into the code base.
> We also incorporated a pre-push commit hook to ensure that no new issues
> are being introduced by committers.
>
> I am glad to announce that we have fixed around *99%* of these issues
> successfully.
> Kudos to everyone involved in this effort !!
>
> --
> Best Regards,
> Suraj Khurana
> Senior Technical Consultant

Reply | Threaded
Open this post in threaded view
|

Re: UPDATE: OFBiz code linting issues

Pritam Kute
Thanks to Suraj and all the other people who worked on this task. This is a
great achievement. The introduction of checkstyle will surely help in
keeping code clean and neat.

Kind Regards,
--
Pritam Kute


On Mon, Sep 28, 2020 at 2:53 PM Jacques Le Roux <
[hidden email]> wrote:

> Thanks Suraj, and to all who worked on it.
>
> I know it's not yet finished, but I think the next similar tasks would be
> to look at the FIXMEs and later TODOs...
>
> Jacques
>
> Le 26/09/2020 à 12:36, Suraj Khurana a écrit :
> > Hello team,
> >
> > Sometimes back, we introduced sonar and checkstyle linting tools
> integrated
> > with OFBiz.
> > We as a community are serious not only about features being introduced in
> > OFBiz but also code quality being added in the repo.
> > So we started correcting issues identified by the linting tool
> (checkstyle)
> > and there were nearly ~*40,000* of those in the whole codebase.
> >
> > Slowly we keep on discussing it on different mail threads and move
> towards
> > conclusions and fix those into the code base.
> > We also incorporated a pre-push commit hook to ensure that no new issues
> > are being introduced by committers.
> >
> > I am glad to announce that we have fixed around *99%* of these issues
> > successfully.
> > Kudos to everyone involved in this effort !!
> >
> > --
> > Best Regards,
> > Suraj Khurana
> > Senior Technical Consultant
>
>