<div dir="ltr">+1 to what Rob said.<div><br></div><div>I guess I don't see what problems is being solved by turning the rule off, and I also don't see the harm in having more check. It does generate a lot of warnings, but invoking `npm run lint -- --quiet` gets rid of those. Also, from my experience, sphinx-based doc builds are notoriously permissive about bad formatting. Eslint's stricter jsdoc checks would add an additional safety net.</div><div><br></div><div>However, if you're working on this with the docs team, then the result should be applicable to the entirety of openstack - meaning that once your work is complete, it may make sense to turn the rule off globally.</div><div><br></div><div>Michael</div><div> </div><div><div><div><div><div class="gmail_quote"><div dir="ltr">On Wed, Mar 9, 2016 at 11:14 AM Rob Cresswell (rcresswe) <<a href="mailto:rcresswe@cisco.com" target="_blank">rcresswe@cisco.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<div style="word-wrap:break-word">
<div>If possible, I’d really prefer we left linting work to Newton. It’ll be good to get it to a more usable state again, but we ought to be focusing on thoroughly checking the new Launch Instance for bugs and edge usage cases, as well as the outstanding
 bugs and blueprints targeted at RC1 (<a href="https://launchpad.net/horizon/+milestone/mitaka-rc1" target="_blank">https://launchpad.net/horizon/+milestone/mitaka-rc1</a>). This is a great opportunity to prove that the Angular rewrites are fully capable of providing
 an improved experience, and we should be capitalising on that.</div></div><div style="word-wrap:break-word">
<div><br>
</div>
<div>Rob</div></div><div style="word-wrap:break-word">
<div><br>
</div>
<br>
<div>
<blockquote type="cite">
<div>On 9 Mar 2016, at 02:25, Richard Jones <<a href="mailto:r1chardj0n3s@gmail.com" target="_blank">r1chardj0n3s@gmail.com</a>> wrote:</div>
<br>
<div>
<div dir="ltr">Hey all,
<div><br>
</div>
<div>I started looking into fixing the wall of "npm run lint" warnings today and quickly noticed that about 85% of the "linting" warnings were about jsdoc. We have significant issues around jsdoc anyway and we're supposed to be using Sphinx anyway[1].</div>
<div><br>
</div>
<div>Some Horizon folks will know that I've been investigating generating publishable documentation for our Javascript code for some time. Most of the tools either don't work (dgeni) or produce pretty unusable output for a project our size (jsdoc and
 grunt-ngdocs). I am about to investigate Sphinx in collaboration with the docs team.</div>
<div><br>
</div>
<div>Regardless, I believe that the documentation generation should generate errors about that documentation, not the code linter. Once we actually get a documentation generator going. Until then, we don't even know what syntax the documentation should
 follow.</div>
<div><br>
</div>
<div>I've proposed a change which just turns jsdoc "linting" off[2]. At the moment, it is less than useful (the noise drowns out any other, legitimate linting).</div>
<div><br>
</div>
<div><br>
</div>
<div>     Richard</div>
<div><br>
</div>
<div><br>
</div>
<div>[1] <a href="http://governance.openstack.org/reference/cti/javascript-cti.html" target="_blank">http://governance.openstack.org/reference/cti/javascript-cti.html</a></div>
<div>[2] <a href="https://review.openstack.org/#/c/290235/" target="_blank">https://review.openstack.org/#/c/290235/</a></div>
<div><br>
</div>
</div>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="mailto:OpenStack-dev-request@lists.openstack.org" target="_blank">
OpenStack-dev-request@lists.openstack.org</a>?subject:unsubscribe<br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div>
</blockquote>
</div>
<br>
</div>

__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div></div></div></div></div></div>