[Engine-devel] IMPORTANT: FindBugs threshold update

Itamar Heim iheim at redhat.com
Wed Jan 1 16:25:19 UTC 2014


On 01/01/2014 04:25 PM, Oved Ourfalli wrote:
> Hi all,
>
> Up until now the jenkins jobs on the gerrit patches included a findbugs job, that failed only in case of a warning of level NORMAL or higher.
> Now we update this threshold to fail on any findbugs warning, including LOW ones.
>
> Please make sure to rebase your current patches and check that the findbugs job finish successfully.
> It will probably fail without a rebase, as the last patches clearing the warnings were merged a few hours ago.
>
> Kodus to everyone involved in clearing all the LOW level warnings... mostly Allon and Alissa, but others helped as well! :-)
>
> Cheers,
> Oved
> _______________________________________________
> Engine-devel mailing list
> Engine-devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/engine-devel
>

indeed - a BIG thank you for cleaning this - we finally have at least 
java naming conventions and various other nits fixed.
lets keep this clean - and that's only possible when its at a clear 
round 0. if something is a false positive, we can add a specific ignore 
for it (which was done for some issues).

Thanks!
    Itamar



More information about the Devel mailing list