activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daniel Kulp <>
Subject Re: [DISCUSS] Artemis Coding Style
Date Tue, 04 Aug 2015 13:51:28 GMT
I’d probably start with the CXF configs:;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.


> On Aug 4, 2015, at 9:45 AM, Clebert Suconic <> 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
> <> 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
>> 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 -
Talend Community Coder -

View raw message