Re: [openstack-dev] [javascript] [horizon] [merlin] [refstack] Javascript Linting

2015-06-17 Thread Thai Q Tran
I agree with Rob in terms of tooling and with Travis on rules.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.

Re: [openstack-dev] [javascript] [horizon] [merlin] [refstack] Javascript Linting

2015-06-16 Thread Michael Krotscheck
-dev] [javascript] [horizon] [merlin] [refstack] Javascript Linting I'm restarting this thread with a different subject line to get a broader audience. Here's the original thread: http://lists.openstack.org/pipermail/openstack-dev/2015-June/066040.html The question at hand is What

Re: [openstack-dev] [javascript] [horizon] [merlin] [refstack] Javascript Linting

2015-06-16 Thread Tripp, Travis S
/master/.jshintrc From: Rob Cresswell (rcresswe) rcres...@cisco.com Reply-To: OpenStack List openstack-dev@lists.openstack.org Date: Tuesday, June 16, 2015 at 1:40 AM To: OpenStack List openstack-dev@lists.openstack.org Subject: Re: [openstack-dev] [javascript] [horizon] [merlin] [refstack

Re: [openstack-dev] [javascript] [horizon] [merlin] [refstack] Javascript Linting

2015-06-16 Thread Michael Krotscheck
On Tue, Jun 16, 2015 at 10:22 AM Tripp, Travis S travis.tr...@hp.com wrote: I think agreeing on rules is the bigger problem here and I don’t think all the projects should have to agree on rules. I believe we agree there, mostly. I personally feel there is some benefit to setting some rules,

Re: [openstack-dev] [javascript] [horizon] [merlin] [refstack] Javascript Linting

2015-06-16 Thread Rob Cresswell (rcresswe)
questions) openstack-dev@lists.openstack.orgmailto:openstack-dev@lists.openstack.org Subject: [openstack-dev] [javascript] [horizon] [merlin] [refstack] Javascript Linting I'm restarting this thread with a different subject line to get a broader audience. Here's the original thread: http

Re: [openstack-dev] [javascript] [horizon] [merlin] [refstack] Javascript Linting

2015-06-15 Thread Richard Jones
JSCS in Horizon has been extended with the John Papa style guidelines to enforce consistent angularjs code style*. It's no longer just a findbug tool. I don't have time to investigate - can ESLint perform the same role for Horizon? Current Horizon activity involves a whole lot of bringing code

Re: [openstack-dev] [javascript] [horizon] [merlin] [refstack] Javascript Linting

2015-06-15 Thread Robert Collins
On 16 June 2015 at 16:21, Richard Jones r1chardj0...@gmail.com wrote: JSCS in Horizon has been extended with the John Papa style guidelines to enforce consistent angularjs code style*. It's no longer just a findbug tool. I don't have time to investigate - can ESLint perform the same role for

Re: [openstack-dev] [javascript] [horizon] [merlin] [refstack] Javascript Linting

2015-06-15 Thread Richard Jones
Sorry, wrong patch. That one added the style requirement to the project contribution guidelines. This is the one that added the .jscsrc config: https://review.openstack.org/#/c/185725/ On Tue, 16 Jun 2015 at 14:21 Richard Jones r1chardj0...@gmail.com wrote: JSCS in Horizon has been extended

[openstack-dev] [javascript] [horizon] [merlin] [refstack] Javascript Linting

2015-06-15 Thread Michael Krotscheck
I'm restarting this thread with a different subject line to get a broader audience. Here's the original thread: http://lists.openstack.org/pipermail/openstack-dev/2015-June/066040.html The question at hand is What will be OpenStack's javascript equivalent of flake8. I'm going to consider the need

Re: [openstack-dev] [javascript] [horizon] [merlin] [refstack] Javascript Linting

2015-06-15 Thread Robert Collins
On 16 June 2015 at 11:36, Michael Krotscheck krotsch...@gmail.com wrote: ... My personal opinion/recommendation: Based on the above, I recommend we use ESLint. My reasoning: It's one tool, it's extensible, it does both codestyle things and bug finding things, and it has a good license. JSHint