-1 for removing checkstyle if the whole project uses the same style its much easier to read.
as said: i would prefer warning for normal builds and enforce failure on release builds for findBugs lg reini > Am 26.01.2015 um 13:17 schrieb Romain Manni-Bucau <[email protected]>: > > difference between checkstyle and findbugs is mainly checkstyle ones > are easy to fix and never false positive > > > Romain Manni-Bucau > @rmannibucau > http://www.tomitribe.com > http://rmannibucau.wordpress.com > https://github.com/rmannibucau > > > 2015-01-26 13:13 GMT+01:00 Oliver B. Fischer <[email protected]>: >> Before we disable or remove it, we can turn it for the default build off. >> But please let us do the same for CheckStyle. It complains only about >> brackets and nothing usefull. >> >> >> Am 26.01.15 um 13:04 schrieb Anatole Tresch: >>> >>> 1) that is an assumption. >>> 2) there is not only black and white: >>> Eg we can add it in failing mode to the delivery build, but use it on >>> warning level for the normal builds. >>> 3) keep in mind there is a relation between code quality and individual >>> skillset. These tools are great, but when their limits start to eat up my >>> freetime with false-positives i switch them off. >>> Oliver B. Fischer <[email protected]> schrieb am Mo., 26. Jan. 2015 >>> um 12:49: >>> >>>> If you set it to warning no one will pay attention... >>>> >>>> Am 26.01.15 um 12:45 schrieb Reinhard Sandtner: >>>> fail - warnings are enough imo but we need to have a much better test >>>> coverage >>>> [email protected]>: >>>> -- >>>> N Oliver B. Fischer >>>> A Schönhauser Allee 64, 10437 Berlin, Deutschland/Germany >>>> P +49 30 44793251 >>>> M +49 178 7903538 >>>> E [email protected] >>>> S oliver.b.fischer >>>> J [email protected] >>>> X http://xing.to/obf >>>> >>>> >> >> -- >> N Oliver B. Fischer >> A Schönhauser Allee 64, 10437 Berlin, Deutschland/Germany >> P +49 30 44793251 >> M +49 178 7903538 >> E [email protected] >> S oliver.b.fischer >> J [email protected] >> X http://xing.to/obf >>
