I’d probably start with the CXF configs: https://git1-us-west.apache.org/repos/asf?p=cxf-build-utils.git;a=tree;f=buildtools/src/main/resources;h=f0a20e01e53e5915a2a298060688c0cf24bc86bf;hb=HEAD
as a base and modify it (likely relax some rules, CXF is pretty strict). The CXF configs are updated for the latest checkstyle plugins for both Maven and the Eclipse checkstyle plugin. I know the Camel configs need some update for the new plugins. Just haven’t gotten around to doing it. Dan > On Aug 4, 2015, at 9:45 AM, Clebert Suconic <[email protected]> wrote: > > +0. I have no problem on switching coding styles as long as we stick > to one and bind it with checkstyle. > > Although, If that will help more people to adopt the codebase and > would help the community, then I am +1000 on that. > > The only favor I ask is if anyone is sending a commit to change > codestyles and reconfigure the checkstyle please do it with a pull > request as that will be used also to validate the PR builds. > > > Maybe we could agree in detail with the checkstyle.xml before we send > a commit doing it though. > > On Tue, Aug 4, 2015 at 7:56 AM, Christopher Shannon > <[email protected]> wrote: >> So in general I'm not too picky with coding styles but I just started >> looking at the Artemis project a few days ago and something that stood out >> to me right away was the use of opening curly braces on a new line. >> >> Virtually ever Java code base I've seen is written in the style using the >> opening brace on the same line. (See >> http://google.github.io/styleguide/javaguide.html#s4.1.2-blocks-k-r-style) >> I think that in general it would be a good idea to match up to a style >> that most Java developers are used to working with if we want to get more >> of the community involved. >> >> I was wondering if anyone would have an issue with changing the style or >> what people's thoughts are about this potential change? > > > > -- > Clebert Suconic -- Daniel Kulp [email protected] - http://dankulp.com/blog Talend Community Coder - http://coders.talend.com
