[GitHub] [beam] kennknowles opened a new issue, #18211: Use checkstyle on everything

[GitBox <gi...@apache.org>]

kennknowles opened a new issue, #18211:
URL: https://github.com/apache/beam/issues/18211

   It is desirable to run checkstyle on everything, including poms.
   
   The problem is that the checkstyle rules come from the classpath, and that you can't have a dependency at the top level on a module of the project.
   
   There are some alternatives:
   
   * As of Maven 3.x, the entire checkstyle rule set can be inlined in the pom. A cure worse than the disease? You decide.
   * What other projects *(e.g. CXF) do is separately release an artifact with checkstyle and PMD rules. You can do that with a new github repo, or using the necessary Maven magic to have a subdirectory which is _not_ a module and contains a releasable pom. (It's all in how you configure the Maven release plugin).
   * You can just arrange for the rules to be at a URL. A URL, for example, to a specific version of a specific file on github?
   
   [~dhalperi@google.com]
   
   Imported from Jira [BEAM-1278](https://issues.apache.org/jira/browse/BEAM-1278). Original Jira may contain additional context.
   Reported by: bmargulies.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org