[Engine-devel] checkstyle and tools project

Allon Mureinik amureini at redhat.com
Tue Jun 25 11:20:16 UTC 2013


checkstyle runs as part of a compile phase in mvn.
as long as you build your code before submitting it to gerrit you should be scott-free.

----- Original Message -----
> From: "Eyal Edri" <eedri at redhat.com>
> To: "Laszlo Hornyak" <lhornyak at redhat.com>
> Cc: "engine-devel" <engine-devel at ovirt.org>, "infra" <infra at ovirt.org>
> Sent: Monday, June 24, 2013 5:28:44 PM
> Subject: Re: [Engine-devel] checkstyle and tools project
> 
> Can we add a gerrit hook to run checkstyle on the code?
> rather than waiting it to be merged and fail on jenkins?
> 
> ----- Original Message -----
> > From: "Laszlo Hornyak" <lhornyak at redhat.com>
> > To: "engine-devel" <engine-devel at ovirt.org>
> > Sent: Monday, June 24, 2013 11:15:38 AM
> > Subject: [Engine-devel] checkstyle and tools project
> > 
> > Hi,
> > 
> > Checkstyle is not executed on tools project. I have sent a patch to gerrit
> > to
> > fix this, could you guys review?
> > 
> > http://gerrit.ovirt.org/15719
> > 
> > And there are some more cleanup patches for tools:
> > http://gerrit.ovirt.org/15720
> > http://gerrit.ovirt.org/15721
> > http://gerrit.ovirt.org/15722
> > 
> > 
> > Thank you,
> > Laszlo
> > _______________________________________________
> > Engine-devel mailing list
> > Engine-devel at ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/engine-devel
> > 
> _______________________________________________
> Engine-devel mailing list
> Engine-devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/engine-devel
> 



More information about the Devel mailing list