the issue I have with checkstyle in compile is that there are many cases where you’re developing and want to build/test with no regard to style but before you would ever commit - e.g. quickly commenting out a test or function, which with checkstyle now throws whitespace errors requiring manual commenting etc. I would rather run style checking tools at the tail end of the development cycle (which I often do via check.pex) before merging vs throughout. to me this feels like it’d be more ideal as a commit hook via a separate

lint

goal - but I know others feel quite differently.