Any idea, with those modifications to checkstyle, how much effort it will take to reformat the code to conform?
On Tue, Feb 21, 2017 at 8:23 AM, Justin Leet <justinjl...@gmail.com> wrote: > As part of: > https://issues.apache.org/jira/browse/METRON-726 > https://github.com/apache/incubator-metron/pull/459 > > I integrated checkstyle into the mvn:site command, and have checkstyle > reports being run as part of the mvn:site reporting. I expect to be > celebrating hitting 25k checkstyle warnings soon. > > I tested out creating a code formatting setup in IntelliJ, with a couple > slight modifications of the default Sun conventions (extended the character > limit of a line past 80 and made it two space indents). Given that > checkstyle includes it as a default option, it's probably reasonably close > to the Sun conventions. I'm thinking we probably also at least create an > Eclipse profile, to open up ease of development. > > There's probably also a discussion about how exactly we want to enforce it. > Is it just something we add to the PR checklist and have reviewers give a > glance, do we setup a hook to autoformat code, etc? > > Justin >