[openstack-dev] [horizon] Javascript development improvement

Imre Farkas ifarkas at redhat.com
Tue Nov 26 12:42:26 UTC 2013


On 11/22/2013 06:13 PM, Julie Pichon wrote:
> "Imre Farkas" <ifarkas at redhat.com> wrote:
>> On 11/22/2013 02:49 PM, Maxime Vidori wrote:
>>>> It seems a bit crazy to me to introduce NodeJS as a dependency just for
>>>> the sake of an easy way to run jslint. There are other options
>>>> available that run without NodeJS as a dependency.
>>>
>>> Sadly, the only solutions which try to implement jslint in pure python are
>>> in alpha or abandoned. If you have a python library which has the same
>>> quality as jslint (which is written by Douglas Crockford himself), I will
>>> be glad to take a look at it.
>>
>> There's a jslint fork called jshint which is able to run in the browser
>> without any node.js dependency.
>>
>> I created a POC patch [1] long time ago to demonstrate its capabilities.
>> It's integrated with qunit and runs automatically with the horizon test
>> suite.
>
> Thanks Imre, this is interesting. Would you mind restoring the patch? If
> you don't have time to work on it please indicate so (I don't think it's
> possible to pick up a patch if the status is 'Abandoned') and someone else
> can look into the test failures.

Julie, Matthias,
I restored the patch, rebased, fixed the tests, added the test files as 
Radomir suggested and removed the .jshintrc file.

The original patch was WIP and I left the failing tests intentionally, 
so that it was easier to compare the two solutions (node.js vs pure 
qunit). But as I wrote, it's cleaned up by now and only contains the latter.

Imre

>
>> The patch also contains a .jshintrc file for the node.js package but you
>> can remove it since it's not used by the qunit+jshint test at all.
>
> Sounds good.
>
> Julie
>
>> Imre
>>
>> [1] https://review.openstack.org/#/c/41087/
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>




More information about the OpenStack-dev mailing list