[openstack-dev] [Tuskar][Horizon] Javascript linter
Monty Taylor
mordred at inaugust.com
Wed Apr 2 18:11:45 UTC 2014
On 04/02/2014 11:05 AM, Ana Krivokapic wrote:
> On 04/02/2014 07:36 PM, Dougal Matthews wrote:
>>
>>
>> On my phone so I can't follow this up more now, but it looks like
>> JSHint is already used.
>>
>> https://review.openstack.org/#/c/41087/
>>
>
> That commit has since been reverted and JSHint removed from Horizon
> codebase:
> https://bugs.launchpad.net/horizon/+bug/1270764
> https://review.openstack.org/#/c/68268/
>
> Seems like there's some controversy around whether the licenses are
> actually incompatible or not. How do we go about clearing that up? :)
>
I was a little unclear before.
Two things:
- We should use the same stuff we've put together for storyboard
- It is not needed to include jshint in the codebase to be able to use it
To expand on the second point - we've found tooling to allow us to
integrate things like jshint into python virtualenv (just go ahead and
be amazed) So it's entirely possible to do _ALL_ of this in a way that
is consistent with how we work in OpenStack. Check out these two patches
to zuul:
https://review.openstack.org/#/c/82307/
https://review.openstack.org/#/c/82308/
Which show incorporating it into a standing python project. If I get a
chance this weekend - I can try to cook up a similar one for
horizon/tuskar if nobody beats me to it.
More information about the OpenStack-dev
mailing list