What about apache-rat and spotbugs? Should those execute by default also?
On Fri, Oct 16, 2020 at 9:29 AM Enrico Olivelli <eolive...@gmail.com> wrote: > > Christopher, > I think it is only a leftover of the adoption of checkstyle. > > Feel free to send a patch and activate it by default > > > Enrico > > Il Ven 16 Ott 2020, 15:25 Christopher <ctubb...@apache.org> ha scritto: > > > Hi ZK Devs, > > > > I am working on ZOOKEEPER-3973 and I noticed the existing Travis CI > > job executes `checkstyle:check` on the command line. I checked the > > pom.xml files and there is an execution defined in the > > `<pluginManagement/>` section of the main pom.xml and another one in > > the zookeeper-contrib/pom.xml. However, neither of these are actually > > activated, as there is no corresponding execution > > `maven-checkstyle-plugin` defined in a `<build><plugins/></build>` > > section, only in the `<pluginManagement/>` section. > > > > So, my question is: what is the intent here? > > > > 1. Should checkstyle always run during a build, unless skipped with > > `-Dcheckstyle.skip` on the command-line? > > > > OR > > > > 2. Should checkstyle always run *only* when explicitly run by placing > > `checkstyle:check` on the command-line? > > > > > > If the answer is 1., then the pom.xml files are missing a > > maven-checkstyle-plugin entry in the `<build><plugins/></build>` > > section and should be added. > > If the answer is 2., then the `<executions/>` block of the > > maven-checkstyle-plugin entry in the `<pluginManagement/>` section is > > doing nothing and should be removed. > > > > Once I know the answer, I can include the change as part of > > ZOOKEEPER-3973, so that way the GitHub Actions CI job I am creating > > for that JIRA is configured to run checkstyle in whatever way the > > project wants, without unnecessary configuration. > > > > Thanks, > > Christopher > >
