16 Apr
2010
16 Apr
'10
9:24 a.m.
Caleb James DeLisle wrote:
I would like to propose changing checkstyle.xml to
allow checkstyle to be enabled and disabled
using inline comments. I propose this with some reservation because this can be a
slippery slope
but I would rather see the problems isolated in the file than the file excluded.
Also I think this can be used to enforce checkstyle on changes made to code in big files
in
the old core.
One problem is that you can easily forget about the pieces of code that
have been excluded from checkstyle. Right now classes that are
excluded/included from/to checkstyle are listed in the project's pom
(those that are excluded should have an explanatory comment, as you
suggested). How can you know which classes still have checkstyle errors
after they were included in the check list? find/grep on the source tree?
Thanks,
Marius
WDYT?
Caleb
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs