[ovirt-devel] Checkstyle changes in oVirt

Martin Perina mperina at redhat.com
Tue Jul 14 20:06:29 UTC 2015


Great work, thanks a lot!

Martin


----- Original Message -----
> From: "Tal Nisan" <tnisan at redhat.com>
> To: "rhev-devel" <rhev-devel at redhat.com>, devel at ovirt.org
> Sent: Tuesday, July 14, 2015 7:11:31 PM
> Subject: [ovirt-devel] Checkstyle changes in oVirt
> 
> Hey everyone,
> 
> We've all reviewed patches that contained imports sorting "noise" caused
> by misconfigured IDEs, I decided to enforce the project's standards
> concerning imports order with the checkstyle plugin.
> Achieving this required upgrading checkstyle to version 6.5 and fixing
> the existing violations so we can have a clean baseline to build from.
> While I was at it, I fixed the left curly braces position suppression in
> the "uicommon" project to align with the proper standards defined in the
> rest of the project.
> 
> Please note that these standards are now enforced at build time and
> patches that violate them will break the build.
> A big thank you goes out to everyone who reviewed and commented on the
> patches!
> 
> * For those interested, the relevant patches can be found at
> https://gerrit.ovirt.org/#/q/status:open+project:ovirt-engine+branch:master+topic:checkstyle
> * The project formatter settings can be found at
> http://www.ovirt.org/Building_oVirt_Engine/IDE
> 
> _______________________________________________
> Devel mailing list
> Devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel
> 



More information about the Devel mailing list