ciena-blueplanet / ember-test-utils

A set of utilities to aid in testing components
MIT License
5 stars 18 forks source link

eslint > 4.13 may break linting scripts #117

Closed twjohnston closed 6 years ago

twjohnston commented 6 years ago

BUG REPORT

Expected Behavior

Linting should complete without error.

Actual Behavior

Linting scripts may throw the following error and exit:

[...]/node_modules/eslint/lib/rules/no-unused-vars.js:410
            if (target.type === "VariableDeclarator") {
                      ^

TypeError: Cannot read property 'type' of undefined

Possible Solution

Pin eslint to 4.13

Steps to Reproduce

Depending on the script, running npm i eslint@4.13.x and then linting should resolve as expected. But, running npm i eslint@4.14.x and then linting may throw the above mentioned error.

Environment

notmessenger commented 6 years ago

It was determined that version 6.0.0 of this repos was being used, so as such this issue is being closed.