Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

yarn lint-js throws #5068

Closed
nchevobbe opened this issue Jan 10, 2018 · 0 comments · Fixed by #5071
Closed

yarn lint-js throws #5068

nchevobbe opened this issue Jan 10, 2018 · 0 comments · Fixed by #5071
Assignees

Comments

@nchevobbe
Copy link
Member

Steps to reproduce:

  1. Pull master
  2. yarn nom
  3. yarn lint-js

Expected results:
the command runs

Actual results:
the command throws :

yarn lint-js v1.0.0-20170810.1438
$ eslint *.js "src/**/*.js" --fix
The babel/generator-star-spacing rule is deprecated. Please use the built in generator-star-spacing rule instead.
Cannot read property 'type' of undefined
TypeError: Cannot read property 'type' of undefined
    at isForInRef (/Users/nchevobbe/Projects/debugger.html/node_modules/eslint/lib/rules/no-unused-vars.js:406:24)
    at variable.references.some.ref (/Users/nchevobbe/Projects/debugger.html/node_modules/eslint/lib/rules/no-unused-vars.js:443:21)
    at Array.some (<anonymous>)
    at isUsedVariable (/Users/nchevobbe/Projects/debugger.html/node_modules/eslint/lib/rules/no-unused-vars.js:442:40)
    at collectUnusedVariables (/Users/nchevobbe/Projects/debugger.html/node_modules/eslint/lib/rules/no-unused-vars.js:565:26)
    at collectUnusedVariables (/Users/nchevobbe/Projects/debugger.html/node_modules/eslint/lib/rules/no-unused-vars.js:572:17)
    at Program:exit (/Users/nchevobbe/Projects/debugger.html/node_modules/eslint/lib/rules/no-unused-vars.js:617:36)
    at listeners.(anonymous function).forEach.listener (/Users/nchevobbe/Projects/debugger.html/node_modules/eslint/lib/util/safe-emitter.js:47:58)
    at Array.forEach (<anonymous>)
    at Object.emit (/Users/nchevobbe/Projects/debugger.html/node_modules/eslint/lib/util/safe-emitter.js:47:38)
    at NodeEventGenerator.applySelector (/Users/nchevobbe/Projects/debugger.html/node_modules/eslint/lib/util/node-event-generator.js:251:26)
    at NodeEventGenerator.applySelectors (/Users/nchevobbe/Projects/debugger.html/node_modules/eslint/lib/util/node-event-generator.js:280:22)
    at NodeEventGenerator.leaveNode (/Users/nchevobbe/Projects/debugger.html/node_modules/eslint/lib/util/node-event-generator.js:303:14)
    at CodePathAnalyzer.leaveNode (/Users/nchevobbe/Projects/debugger.html/node_modules/eslint/lib/code-path-analysis/code-path-analyzer.js:630:23)
    at Traverser.leave [as _leave] (/Users/nchevobbe/Projects/debugger.html/node_modules/eslint/lib/linter.js:1001:32)
    at Traverser._traverse (/Users/nchevobbe/Projects/debugger.html/node_modules/eslint/lib/util/traverser.js:155:18)
error Command failed with exit code 1.

Could be related to eslint/eslint#9767

@claim claim bot added the not-available label Jan 10, 2018
@nchevobbe nchevobbe self-assigned this Jan 10, 2018
nchevobbe added a commit to nchevobbe/debugger.html that referenced this issue Jan 10, 2018
This fixes firefox-devtools#5068.

Took this as an opportunity to fix the warning message about deprecated eslint rule.
nchevobbe added a commit to nchevobbe/debugger.html that referenced this issue Jan 10, 2018
This fixes firefox-devtools#5068.

Took this as an opportunity to fix the warning message about deprecated eslint rule.
nchevobbe added a commit to nchevobbe/debugger.html that referenced this issue Jan 10, 2018
This fixes firefox-devtools#5068.

Took this as an opportunity to fix the warning message about deprecated eslint rule.
nchevobbe added a commit that referenced this issue Jan 10, 2018
This fixes #5068.

Took this as an opportunity to fix the warning message about deprecated eslint rule.
jasonLaster pushed a commit that referenced this issue Jan 10, 2018
This fixes #5068.

Took this as an opportunity to fix the warning message about deprecated eslint rule.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
1 participant