<br><br>On Tuesday, May 20, 2014, Bruno Oliveira <<a href="mailto:bruno@abstractj.org">bruno@abstractj.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The only reason to have checkstyle enabled by default is: if we agree on<br>
which rules should be active or not and provide an specific IDE setup.</blockquote><div><br></div><div>Exactly! (Includes headers, formarting etc)</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Other than that, people like me will skip it. Why? Simple, I'm trying to<br>
solve critical problems and also strugging to figure out why checkstyle<br>
is do care about method lenght.</blockquote><div><br></div><div>Same here :-)</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
So to me if you're guys really want to introduce it we need:<br>
<br>
- Definition of which rules were supposed to be active<br>
- IDE profiles for Eclipse/IntelliJ<br>
- Make the error messages something clear</blockquote><div><br></div><div>+1</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Otherwise, I'm -∞. It was never discussed here and if it exists on<br>
aerogear-parent is all our fault.</blockquote><div><br></div><div>Yep, fully agree<span></span></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On 2014-05-20, Daniel Bevenius wrote:<br>
> -1 I'd prefer to have checkstyle enabled by default, and integrate the<br>
> checkstyle into the IDE to avoid having to discover issue later when<br>
> building with maven.<br>
><br>
><br>
> On 20 May 2014 20:56, Bruno Oliveira <<a href="javascript:;" onclick="_e(event, 'cvml', 'bruno@abstractj.org')">bruno@abstractj.org</a>> wrote:<br>
><br>
> > +1 on having an specific profile for checkstyle<br>
> ><br>
> > On 2014-05-20, Matthias Wessendorf wrote:<br>
> > > Hello,<br>
> > ><br>
> > > w/ the advent of the 0.2.0 parent, we have checkstyle enabled;<br>
> > ><br>
> > > However, I am not that happy, as the default rules are IMO a bit odd<br>
> > (e.g.<br>
> > > the unused imports is pretty nasty when developing)<br>
> > ><br>
> > > We could:<br>
> > > a) get rid of it (perhaps not)<br>
> > > b) disable it on normal execution and only execute it on a release<br>
> > profile<br>
> > > or like that<br>
> > ><br>
> > > right now I am running w/ skip - but that's a bit nasty...<br>
> > ><br>
> > > Thoughts?<br>
> > > Matthias<br>
> > ><br>
> > > --<br>
> > > Matthias Wessendorf<br>
> > ><br>
> > > blog: <a href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>
> > > sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>
> > > twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a><br>
> ><br>
> > > _______________________________________________<br>
> > > aerogear-dev mailing list<br>
> > > <a href="javascript:;" onclick="_e(event, 'cvml', 'aerogear-dev@lists.jboss.org')">aerogear-dev@lists.jboss.org</a><br>
> > > <a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br>
> ><br>
> ><br>
> > --<br>
> ><br>
> > abstractj<br>
> > _______________________________________________<br>
> > aerogear-dev mailing list<br>
> > <a href="javascript:;" onclick="_e(event, 'cvml', 'aerogear-dev@lists.jboss.org')">aerogear-dev@lists.jboss.org</a><br>
> > <a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br>
> ><br>
<br>
> _______________________________________________<br>
> aerogear-dev mailing list<br>
> <a href="javascript:;" onclick="_e(event, 'cvml', 'aerogear-dev@lists.jboss.org')">aerogear-dev@lists.jboss.org</a><br>
> <a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br>
<br>
<br>
--<br>
<br>
abstractj<br>
_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="javascript:;" onclick="_e(event, 'cvml', 'aerogear-dev@lists.jboss.org')">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a></blockquote><br><br>-- <br>Sent from Gmail Mobile<br>