Excellent. I also like the idea of putting the configuration in each repository. I have a few pull requests that now do this for the various repositories.
In the pull requests for the -extensions and -sandbox repositories, I put the checkstyle configuration into a ./buildtools directory that isn't a maven project at all, so there is no way it can be deployed or installed as an artifact. This structure also makes the configuration easier for supporting the maven-site-plugin (i.e. mvn site). Regards, Aaron > On Jan 18, 2019, at 2:08 AM, P. Ottlinger <pottlin...@apache.org> wrote: > > Am 17.01.19 um 17:02 schrieb Aaron Coburn: >> What would you all think about adjusting the maven configuration to have >> checkstyle run on each build? Initially, it might make sense to just report >> the errors rather than failing an entire build because of them. I can supply >> a PR for this. > > cool - I'm in favour of checking but would put the configuration in each > of our repository and would not deploy it as an artifact. > > Thanks, > Phil
