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

/**/*.js file matching issue in package.json #320

Closed
digitalsadhu opened this issue Nov 6, 2015 · 1 comment

Comments

@digitalsadhu
Copy link

commented Nov 6, 2015

I can't work this out and it may well not be anything to do with standard but I can run:

./node_modules/.bin/standard server/**/*.js

And detect errors in server/index.js

However the exact same command when run from npm script detects no errors in server/index.js

...
"lint": "./node_modules/.bin/standard server/**/*.js",
...
@feross

This comment has been minimized.

Copy link
Member

commented Nov 21, 2015

I couldn't reproduce this. What version of standard are you running? You can check with standard --version. If you're not running the latest (5.4.1), you should update.

Pretty sure that this bug was fixed a long time ago. If you're still experiencing this on the latest version of standard, please comment and let us know.


Also, unrelated, but you can simplify your npm run scripts by omitting ./node_modules/./bin since npm run automatically adds everything in that folder to your path. So this would do the same thing:

...
"lint": "standard server/**/*.js",
...

@feross feross closed this Nov 21, 2015

@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.