This commit applies the quotes rule to keep the consistency. It seems we
and the Google style guide prefer single-quotes over double-quotes.
That's the reason why I choose single-quotes.
I don't want to stick this kind of things, actually. But I often confuse
that 'which quotes should I use?'. This commit avoids it.
Change-Id: Ic686d7e6bc91d8b75bba033c0b265b48a522df44
This disables the `one-var` eslint rule as it appears to be be
incorrectly raising an error when multiple `for ... in` loops are
used within a single function.
Change-Id: I4043e6656c57ed95e4e359107179261c368340cd
This commit fixes the errors raised by eslint using openstack
configurations.
Change-Id: I3aa0c91e77f40c6383ac73850f5b93ebae9af323
Co-Authored-By: Caio Carrara <ccarrara@thoughtworks.com>
Both jshint and jslint contain the 'do no evil' license which
makes it unusable for OpenStack. This patch switches this project
to use eslint, the published eslint-config-openstack rules, and
the john-papa eslint style guides. It also adds the 'npm run lint'
command used by infra in the javascript-jobs macro.
Change-Id: I724d3a12fb5b9c85446adcd07c03a676e966bd41