-
Notifications
You must be signed in to change notification settings - Fork 30
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
Failing to load eslint #57
Comments
Hello @rplopes, it's linked with the upgrade to npm3, since npm3 flatten node directories I did not came up with a fix right know but I'm looking forward to it |
Hi @FGRibreau. Thank you for the reply. However, I'm still using npm 2:
Is this behaviour to be expected with this version of npm? |
Is |
Yes, and running it from the
Removing the |
We just need to update check-build eslint.js with |
We've been using
check-build
to runjscs
andjshint
. We now wanted to addeslint
, butcheck-build
fails to load it (runningeslint
separately works fine).Simplified
package.json
:Our
.checkbuild
after addingeslint
:Running
eslint
independently works:However,
check-build
fails oneslint
:I don't fully understand if this is an issue with
--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28717323-failing-to-load-eslint?utm_campaign=plugin&utm_content=tracker%2F7304231&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F7304231&utm_medium=issues&utm_source=github).check-build
looking foreslint
in the wrong directory (if it was meant to use <project_dir>/node_modules/.bin/eslint instead, which exists) or looking at the right directory but for some reason not being shipped with the necessary bin.The text was updated successfully, but these errors were encountered: