dequelabs / axe-cli

[Deprecated] A command-line interface for the aXe accessibility testing engine
Mozilla Public License 2.0
430 stars 35 forks source link

chore: fix ESLint errors #90

Closed stephenmathieson closed 5 years ago

stephenmathieson commented 5 years ago

This patch removes an ESLint directive which declared mocha as a global variable. Since we never referenced the global, ESLint was erroring with:

/home/circleci/axe-cli/test/axe-test-urls.js
  1:11  error  'mocha' is defined but never used  no-unused-vars

/home/circleci/axe-cli/test/integrations.js
  1:11  error  'mocha' is defined but never used  no-unused-vars

/home/circleci/axe-cli/test/webdriver.js
  1:11  error  'mocha' is defined but never used  no-unused-vars

✖ 3 problems (3 errors, 0 warnings)

Additionally, the lint script now explicitly lists which files it "lints", rather than assuming ESLint is going to do what we want it to.

Reviewer checks

Required fields, to be filled out by PR reviewer(s)