And if we want to do some painting of the bike shed this is how headers should look like, I've tried to find the header style used by most files in order to change the least number of files:
For Java/Groovy/Scala: https://github.com/zregvart/camel/blob/5c944919d3e9bc22e17fcec20eb91ea36f85b27c/buildingtools/src/main/resources/header-java.txt For properties files: https://github.com/zregvart/camel/blob/5c944919d3e9bc22e17fcec20eb91ea36f85b27c/buildingtools/src/main/resources/header-properties.txt For XML files: https://github.com/zregvart/camel/blob/5c944919d3e9bc22e17fcec20eb91ea36f85b27c/buildingtools/src/main/resources/header-xml.txt The Camel POM now includes the definition of mycila Maven license plugin that can be used to automatically format the headers with `mvn -Pupdate-license` zoran On Fri, May 5, 2017 at 12:32 PM, Zoran Regvart <[email protected]> wrote: > Hi Cameleers, > I'll merge (probably tomorrow), the PR[1] for CAMEL-11194[2], this > upgrades Checkstyle version and changes the Checkstyle rules to > include header checking from Java, Groovy, Scala, properties, XML and > XSD files. > > I've rewriten some of the headers to pass the checks and fixed a small > number of tests that were influenced by this. > > You can see all of this if you checkout PR#1656[1]. > > I think that the Checkstyle upgrade was long overdue, and went down > the rabbit hole of re-formatting the headers -- so I bundled all of > this in one go. > > A few pointers, for the weary ones that wish to review this: > > I first fixed a small number of Checkstyle issues: > https://github.com/apache/camel/pull/1656/commits/462c2b28b6c9a5f856f985c0c6d5947640530c39 > > Then I commited the upgraded Checkstyle version and rule changes: > https://github.com/apache/camel/pull/1656/commits/5c944919d3e9bc22e17fcec20eb91ea36f85b27c > > Changed a great number of headers (hopefully not needed for review): > https://github.com/apache/camel/pull/1656/commits/78f0baa629b61cdc3cd0d290814669a79ac98b2c > > Fixed tests that were affected by the change: > https://github.com/apache/camel/pull/1656/commits/126015ba92d75016a4fb9891d8b575a5ef7e19b2 > > zoran > > [1] https://github.com/apache/camel/pull/1656 > [2] https://issues.apache.org/jira/browse/CAMEL-11194 > -- > Zoran Regvart -- Zoran Regvart
