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

Fix the installation of cli. #53

Merged
merged 1 commit into from
Nov 10, 2017
Merged

Conversation

michelts
Copy link
Contributor

Hello, @cheton

I'm trying to install the latest release of i18next-scanner without success:

$ npm install i18next-scanner
...
npm ERR! enoent ENOENT: no such file or directory, chmod '/tmp/container/node_modules/i18next-scanner/bin/cli.js'
npm ERR! enoent ENOENT: no such file or directory, chmod '/tmp/container/node_modules/i18next-scanner/bin/cli.js'
npm ERR! enoent This is most likely not a problem with npm itself
npm ERR! enoent and is related to npm not being able to find a file.
npm ERR! enoent 

I could install it if I ignore the bin links:

$ npm install i18next-scanner --no-bin-links

I think you miss the bin folder to be added to the package :)

I added it but I don't know if we need to test it, or how to test it. Do you have any suggestion?

@coveralls
Copy link

coveralls commented Nov 10, 2017

Coverage Status

Coverage remained the same at 91.877% when pulling fca8be1 on michelts:fix-cli-installation into f81bd6d on i18next:master.

@cheton cheton merged commit 803e501 into i18next:master Nov 10, 2017
@cheton
Copy link
Member

cheton commented Nov 10, 2017

Arrgh. I forgot adding "bin" to the files. Thank you for the PR.

@michelts
Copy link
Contributor Author

No problem :)

Thank you for the quick response!

@cheton
Copy link
Member

cheton commented Nov 10, 2017

Published v2.1.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants