<br><br>On Tuesday, May 20, 2014, Bruno Oliveira &lt;<a href="mailto:bruno@abstractj.org">bruno@abstractj.org</a>&gt; 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&#39;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&#39;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&#39;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>
&gt; -1 I&#39;d prefer to have checkstyle enabled by default, and integrate the<br>
&gt; checkstyle into the IDE to avoid having to discover issue later when<br>
&gt; building with maven.<br>
&gt;<br>
&gt;<br>
&gt; On 20 May 2014 20:56, Bruno Oliveira &lt;<a href="javascript:;" onclick="_e(event, &#39;cvml&#39;, &#39;bruno@abstractj.org&#39;)">bruno@abstractj.org</a>&gt; wrote:<br>
&gt;<br>
&gt; &gt; +1 on having an specific profile for checkstyle<br>
&gt; &gt;<br>
&gt; &gt; On 2014-05-20, Matthias Wessendorf wrote:<br>
&gt; &gt; &gt; Hello,<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; w/ the advent of the 0.2.0 parent, we have checkstyle enabled;<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; However, I am not that happy, as the default rules are IMO a bit odd<br>
&gt; &gt; (e.g.<br>
&gt; &gt; &gt; the unused imports is pretty nasty when developing)<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; We could:<br>
&gt; &gt; &gt; a) get rid of it (perhaps not)<br>
&gt; &gt; &gt; b) disable it on normal execution and only execute it on a release<br>
&gt; &gt; profile<br>
&gt; &gt; &gt; or like that<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; right now I am running w/ skip - but that&#39;s a bit nasty...<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; Thoughts?<br>
&gt; &gt; &gt; Matthias<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; --<br>
&gt; &gt; &gt; Matthias Wessendorf<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; blog: <a href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>
&gt; &gt; &gt; sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>
&gt; &gt; &gt; twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a><br>
&gt; &gt;<br>
&gt; &gt; &gt; _______________________________________________<br>
&gt; &gt; &gt; aerogear-dev mailing list<br>
&gt; &gt; &gt; <a href="javascript:;" onclick="_e(event, &#39;cvml&#39;, &#39;aerogear-dev@lists.jboss.org&#39;)">aerogear-dev@lists.jboss.org</a><br>
&gt; &gt; &gt; <a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt; --<br>
&gt; &gt;<br>
&gt; &gt; abstractj<br>
&gt; &gt; _______________________________________________<br>
&gt; &gt; aerogear-dev mailing list<br>
&gt; &gt; <a href="javascript:;" onclick="_e(event, &#39;cvml&#39;, &#39;aerogear-dev@lists.jboss.org&#39;)">aerogear-dev@lists.jboss.org</a><br>
&gt; &gt; <a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br>
&gt; &gt;<br>
<br>
&gt; _______________________________________________<br>
&gt; aerogear-dev mailing list<br>
&gt; <a href="javascript:;" onclick="_e(event, &#39;cvml&#39;, &#39;aerogear-dev@lists.jboss.org&#39;)">aerogear-dev@lists.jboss.org</a><br>
&gt; <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, &#39;cvml&#39;, &#39;aerogear-dev@lists.jboss.org&#39;)">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>