<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 23, 2014 at 10:35 AM, Bolesław Dawidowicz <span dir="ltr">&lt;<a href="mailto:bdawidow@redhat.com" target="_blank">bdawidow@redhat.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 05/22/2014 01:50 PM, Matthias Wessendorf wrote:<br>
&gt;<br>
&gt;<br>
&gt; On Thursday, May 22, 2014, Karel Piwko &lt;<a href="mailto:kpiwko@redhat.com">kpiwko@redhat.com</a><br>
</div><div class="">&gt; &lt;mailto:<a href="mailto:kpiwko@redhat.com">kpiwko@redhat.com</a>&gt;&gt; wrote:<br>
&gt;<br>
&gt;     This is the same issue as with unit tests. If they tend to take too<br>
&gt;     much time, people are skipping them.<br>
&gt;<br>
&gt;     What about activating checkstyle in CI and or Git push hook, for<br>
&gt;     both PRs and<br>
&gt;     direct commits upstream. This way a developer can develop without<br>
&gt;     constraints<br>
&gt;     *until* he wants to push the code upstream. Would it help?<br>
&gt;<br>
&gt;<br>
&gt; -1<br>
&gt;<br>
&gt; I think we should work on proper templates for our IDEs instead<br>
<br>
<br>
</div>Major issue with IDE templates is that you cannot get both IDEA and<br>
Eclipse to reformat code in whole project into exactly same outcome.<br>
Unless you force people to use Eclipse formatter plugin in IDEA.<br>
<br>
Also... checkstyle maven plugin is buggy... :)<br></blockquote><div><br></div><div>sounds like you love checkstyle too :) </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5"><br>
&gt;<br>
&gt;<br>
&gt;     Thanks,<br>
&gt;<br>
&gt;     Karel<br>
&gt;<br>
&gt;     On Wed, 21 May 2014 11:38:02 -0300<br>
&gt;     Bruno Oliveira &lt;<a href="mailto:bruno@abstractj.org">bruno@abstractj.org</a>&gt; wrote:<br>
&gt;<br>
&gt;      &gt; I don&#39;t think you need to prove anything, to me is just the matter of<br>
&gt;      &gt; reach a consensus and think about what works for the whole team.<br>
&gt;      &gt;<br>
&gt;      &gt; Is still possible skip checkstyle and leave the mess to the<br>
&gt;     person doing the<br>
&gt;      &gt; release, even worse, the person doing the release can also skip this.<br>
&gt;      &gt;<br>
&gt;      &gt; The only thing able to guarantee the quality of the code is the<br>
&gt;     developer&#39;s<br>
&gt;      &gt; conscience. I don&#39;t think that enforcing checkstyle default rules<br>
&gt;     will make<br>
&gt;      &gt; us more conscious tbh.<br>
&gt;      &gt;<br>
&gt;      &gt; On 2014-05-21, Karel Piwko wrote:<br>
&gt;      &gt; &gt; That&#39;s not true, it was discussed during team meeting at<br>
&gt;      &gt; &gt; the times we&#39;ve been using G+ as well (I can&#39;t prove that) and<br>
&gt;     on ML as<br>
&gt;      &gt; &gt; well:<br>
&gt;      &gt; &gt;<br>
&gt;      &gt; &gt;<br>
&gt;     <a href="http://aerogear-dev.1069024.n5.nabble.com/aerogear-dev-JavaScript-formatter-td5200.html" target="_blank">http://aerogear-dev.1069024.n5.nabble.com/aerogear-dev-JavaScript-formatter-td5200.html</a><br>
&gt;      &gt; &gt;<br>
&gt;      &gt; &gt; It just took 7 month for the PR to be merged. The point of initial<br>
&gt;      &gt; &gt; checkstyle rules set was chosen because:<br>
&gt;      &gt; &gt;<br>
&gt;      &gt; &gt; * It was identified as the least annoying setup on ML, there<br>
&gt;     were a lot of<br>
&gt;      &gt; &gt; +1s<br>
&gt;      &gt; &gt; * There was a bug in released library caused by using unused<br>
&gt;     import that<br>
&gt;      &gt; &gt; was not available on classpath. I might be able to dig it out<br>
&gt;     but it would<br>
&gt;      &gt; &gt; take me a lot of time as I don&#39;t recall what project it was.<br>
&gt;      &gt; &gt;<br>
&gt;      &gt; &gt; +9001 for abstract&#39;s proposal on having same setup for IDE as<br>
&gt;     well. There is<br>
&gt;      &gt; &gt; nothing more annoying then something working in IDE but not in<br>
&gt;     Maven. -1<br>
&gt;      &gt; &gt; for the specific profile, as it shifts responsibility to clean<br>
&gt;     up the mess<br>
&gt;      &gt; &gt; to the person doing the release.<br>
&gt;      &gt; &gt;<br>
&gt;      &gt; &gt; Karel<br>
&gt;      &gt; &gt;<br>
&gt;      &gt; &gt; On Tue, 20 May 2014 16:20:52 -0300<br>
&gt;      &gt; &gt; Bruno Oliveira &lt;<a href="mailto:bruno@abstractj.org">bruno@abstractj.org</a>&gt; wrote:<br>
&gt;      &gt; &gt;<br>
&gt;      &gt; &gt; &gt; The only reason to have checkstyle enabled by default is: if<br>
&gt;     we agree on<br>
&gt;      &gt; &gt; &gt; which rules should be active or not and provide an specific<br>
&gt;     IDE setup.<br>
&gt;      &gt; &gt; &gt;<br>
&gt;      &gt; &gt; &gt; Other than that, people like me will skip it. Why? Simple,<br>
&gt;     I&#39;m trying to<br>
&gt;      &gt; &gt; &gt; solve critical problems and also strugging to figure out why<br>
&gt;     checkstyle<br>
&gt;      &gt; &gt; &gt; is do care about method lenght.<br>
&gt;      &gt; &gt; &gt;<br>
&gt;      &gt; &gt; &gt; So to me if you&#39;re guys really want to introduce it we need:<br>
&gt;      &gt; &gt; &gt;<br>
&gt;      &gt; &gt; &gt; - Definition of which rules were supposed to be active<br>
&gt;      &gt; &gt; &gt; - IDE profiles for Eclipse/IntelliJ<br>
&gt;      &gt; &gt; &gt; - Make the error messages something clear<br>
&gt;      &gt; &gt; &gt;<br>
&gt;      &gt; &gt; &gt; Otherwise, I&#39;m -∞. It was never discussed here and if it<br>
&gt;     exists on<br>
&gt;      &gt; &gt; &gt; aerogear-parent is all our fault.<br>
&gt;      &gt; &gt; &gt;<br>
&gt;      &gt; &gt; &gt; On 2014-05-20, Daniel Bevenius wrote:<br>
&gt;      &gt; &gt; &gt; &gt; -1 I&#39;d prefer to have checkstyle enabled by default, and<br>
&gt;     integrate the<br>
&gt;      &gt; &gt; &gt; &gt; checkstyle into the IDE to avoid having to discover issue<br>
&gt;     later when<br>
&gt;      &gt; &gt; &gt; &gt; building with maven.<br>
&gt;      &gt; &gt; &gt; &gt;<br>
&gt;      &gt; &gt; &gt; &gt;<br>
&gt;      &gt; &gt; &gt; &gt; On 20 May 2014 20:56, Bruno Oliveira &lt;<a href="mailto:bruno@abstractj.org">bruno@abstractj.org</a>&gt;<br>
&gt;     wrote:<br>
&gt;      &gt; &gt; &gt; &gt;<br>
&gt;      &gt; &gt; &gt; &gt; &gt; +1 on having an specific profile for checkstyle<br>
&gt;      &gt; &gt; &gt; &gt; &gt;<br>
&gt;      &gt; &gt; &gt; &gt; &gt; On 2014-05-20, Matthias Wessendorf wrote:<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt; Hello,<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt;<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt; w/ the advent of the 0.2.0 parent, we have checkstyle<br>
&gt;     enabled;<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt;<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt; However, I am not that happy, as the default rules are<br>
&gt;     IMO a bit odd<br>
&gt;      &gt; &gt; &gt; &gt; &gt; (e.g.<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt; the unused imports is pretty nasty when developing)<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt;<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt; We could:<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt; a) get rid of it (perhaps not)<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt; b) disable it on normal execution and only execute it<br>
&gt;     on a release<br>
&gt;      &gt; &gt; &gt; &gt; &gt; profile<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt; or like that<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt;<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt; right now I am running w/ skip - but that&#39;s a bit nasty...<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt;<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt; Thoughts?<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt; Matthias<br>
&gt;      &gt; &gt; &gt; &gt; &gt; &gt;<br>
&gt;      &gt; &gt; &gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; --<br>
&gt; Sent from Gmail Mobile<br>
&gt;<br>
&gt;<br>
</div></div><div class="im HOEnZb">&gt; _______________________________________________<br>
&gt; aerogear-dev mailing list<br>
&gt; <a href="mailto:aerogear-dev@lists.jboss.org">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>
&gt;<br>
<br>
<br>
<br>
--<br>
</div><span class="HOEnZb"><font color="#888888">Bolesław Dawidowicz<br>
JBoss Portal Platform Architect | GateIn Portal Project Lead<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto: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></div></div></blockquote></div><br><br clear="all"><div><br></div>-- <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>
</div></div>