<font face="Default Sans Serif,Verdana,Arial,Helvetica,sans-serif" size="2">I agree with Rob in terms of tooling and with Travis on rules.<div><br></div><div>Any linting tool is fine with me as long as it does not break the rules we currently have set in Horizon. I believe the rules right now are general enough, so switching to eslint might not be an issue, but its something to look into. And we are still early in the JSCS stages, so this is actually opportune timing. I don't have a strong opinion on which tool so as long as it gets the job done and does not stall work in Horizon for liberty.<div><br><font color="#990099">-----Michael Krotscheck <krotscheck@gmail.com> wrote: -----</font><div class="iNotesHistory" style="padding-left:5px;"><div style="padding-right:0px;padding-left:5px;border-left:solid black 2px;">To: "OpenStack Development Mailing List (not for usage questions)" <openstack-dev@lists.openstack.org><br>From: Michael Krotscheck <krotscheck@gmail.com><br>Date: 06/16/2015 12:30PM<br>Subject: Re: [openstack-dev] [javascript] [horizon] [merlin] [refstack] Javascript Linting<br><br><div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Jun 16, 2015 at 10:22 AM Tripp, Travis S <<a href="mailto:travis.tripp@hp.com">travis.tripp@hp.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I think agreeing on rules is the bigger problem here and I don’t think all<br> the projects should have to agree on rules.</blockquote><div><br></div><div>I believe we agree there, mostly. I personally feel there is some benefit to setting some rules, likely published as an openstack linting plugin, which enforce things like "Do not use fuzzy versions in your package.json" and other things that make things unstable. That should be a very carefully reserved list of rules though.</div><div><br></div><div>I've created an eslint configuration file that includes every single rule, it's high level purpose, and a link to the details on it, and provided it in a patch against horizon. The intent is that it's a good starting point from which to activate and deactivate rules that make sense for horizon.</div><div><br></div><div><a href="https://review.openstack.org/#/c/192327/">https://review.openstack.org/#/c/192327/</a><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">We’ve spent a good portion of liberty 1 getting the code base cleaned up to meet the already adopted horizon rules and it is still in progress.<br></blockquote><div><br></div><div>As a side note, the non-voting horizon linting job for javascript things is waiting for review here: <a href="https://review.openstack.org/#/c/188886/">https://review.openstack.org/#/c/188886/</a></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">My preference would be to see if we can use eslint to accomplish all of<br> our currently adopted horizon rules [3][4] AND to also add in the angular<br> specific plugin [1][2]. But we can’t do this at the expense of the entire<br> liberty release.<br></blockquote><div><br></div><div>Again, I agree. The patch I've provided above sets up the horizon eslint build, and adds about... 10K additional style violations. Since neither of the builds pass, it's difficult to see the difference, yet either way you should probably tweak the rules to match horizon's personal preferences.</div><div><br></div><div>Michael</div></div></div> <div><font face="Courier New,Courier,monospace" size="3">__________________________________________________________________________<br>OpenStack Development Mailing List (not for usage questions)<br>Unsubscribe: OpenStack-dev-request@lists.openstack.org?subject:unsubscribe<br><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br></font></div></div></div></div></div></font>