Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unexpected linter output #950

Closed
michaelnisi opened this issue Jul 12, 2017 · 2 comments

Comments

@michaelnisi
Copy link

commented Jul 12, 2017

Today, trying the latest and greatest: Node v8.1.3, npm 5.2.0, standard 10.0.2, I got standard: Unexpected linter output, while running standard via npm pretest script. This might as well be a npm issue, but I decided to put it here first. Running standard alone works as expected, local and global versions of standard are 10.0.2.

  "scripts": {
    "pretest": "standard", ...

This never failed me, but today, after updating npm and standard, I got:

± % npm test                                                                                                                                                      !10018

> pickup@5.1.1 pretest /Users/michael/workspace/pickup
> standard

standard: Unexpected linter output:

Error: Failed to load plugin import: Cannot find module 'eslint-plugin-import'
Referenced from: /Users/michael/workspace/pickup/node_modules/standard/eslintrc.json
    at Function.Module._resolveFilename (module.js:485:15)
    at Function.resolve (internal/module.js:18:19)
    at Object.load (/Users/michael/workspace/pickup/node_modules/eslint/lib/config/plugins.js:134:29)
    at Array.forEach (native)
    at Object.loadAll (/Users/michael/workspace/pickup/node_modules/eslint/lib/config/plugins.js:162:21)
    at load (/Users/michael/workspace/pickup/node_modules/eslint/lib/config/config-file.js:541:21)
    at configExtends.reduceRight (/Users/michael/workspace/pickup/node_modules/eslint/lib/config/config-file.js:424:36)
    at Array.reduceRight (native)
    at applyExtends (/Users/michael/workspace/pickup/node_modules/eslint/lib/config/config-file.js:408:28)
    at Object.load (/Users/michael/workspace/pickup/node_modules/eslint/lib/config/config-file.js:566:22)

If you think this is a bug in `standard`, open an issue: https://github.com/feross/standard/issues
npm ERR! Test failed.  See above for more details.
@michaelnisi

This comment has been minimized.

Copy link
Author

commented Jul 12, 2017

Pointing pretest into node_modules doesn’t make a difference, resulting in the same error.

 "scripts": {
    "pretest": "./node_modules/standard/bin/cmd.js", ...
@LinusU

This comment has been minimized.

Copy link
Member

commented Jul 12, 2017

This was a breaking change in npm v5.2.0 that has now been reverted. Use npm v5.1.0 as a workaround until v5.2.1 is available.

Duplicate of #949

@LinusU LinusU closed this Jul 12, 2017

@lock lock bot locked as resolved and limited conversation to collaborators May 10, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
2 participants
You can’t perform that action at this time.