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 [email protected] http://lists.xwiki.org/mailman/listinfo/devs