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

npm run lint throws error when spaces are in path on Windows #150

Closed
alvipeo opened this issue May 31, 2016 · 4 comments
Closed

npm run lint throws error when spaces are in path on Windows #150

alvipeo opened this issue May 31, 2016 · 4 comments
Labels

Comments

@alvipeo
Copy link

alvipeo commented May 31, 2016

Node version:
5.1.11
npm version:
3.8.6

The output:

> react-slingshot@4.0.0 lint d:\Documents\Visual Studio 2015\Projects\NewsNotifier
> esw webpack.config.* src tools

'd:\Documents\Visual' is not recognized as an internal or external command,
operable program or batch file.

npm ERR! Windows_NT 10.0.10586
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run" "lint"
npm ERR! node v5.11.1
npm ERR! npm  v3.8.6
npm ERR! code ELIFECYCLE
npm ERR! react-slingshot@4.0.0 lint: `esw webpack.config.* src tools`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the react-slingshot@4.0.0 lint script 'esw webpack.config.* src tools'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the react-slingshot package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     esw webpack.config.* src tools
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs react-slingshot
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls react-slingshot
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     d:\Documents\Visual Studio 2015\Projects\NewsNotifier\npm-debug.log
@coryhouse
Copy link
Owner

coryhouse commented May 31, 2016

Ah! It's because you have a space in your path. eslint-watch doesn't support running in a path with spaces. I'll open a ticket against them, but in the meantime I'd suggest moving your project to a path that doesn't contain spaces.

@alvipeo
Copy link
Author

alvipeo commented May 31, 2016

Thanks for filing the issue.

@coryhouse coryhouse changed the title npm run lint - error on Windows 10 npm run lint - error on Windows Jun 9, 2016
@coryhouse coryhouse added the bug label Jun 9, 2016
@coryhouse
Copy link
Owner

Just an update - eslint-watch now has a branch with this issue fixed so can hopefully close this soon.

@coryhouse coryhouse changed the title npm run lint - error on Windows npm run lint throws error when spaces are in path on Windows Jun 28, 2016
@coryhouse
Copy link
Owner

Just updated to the latest version of eslint-watch to resolve this.

davidclough added a commit to davidclough/node that referenced this issue Feb 27, 2018
…n execute "npm run lint" when there are spaces in the project path.

The previously used version (2.1.11) error when the command was executed. Error was indicated at coryhouse/react-slingshot#150
Once this was done we also saw "npm run lint:watch" actually run the tests initially. Previously it only ran tests when on files that had been changed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants