-
-
Notifications
You must be signed in to change notification settings - Fork 425
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
[feature request]: Don't show install message if lint-staged is already configured #192
Comments
Totally agree! It would be awesome if you could create a PR that implements this. |
It seems like you dont actually do any install. its just a greeting message. https://github.com/okonet/lint-staged/blob/master/package.json#L9 |
its confusing |
I think you're right and it's worth removing it. The only concern I have is that people who will install it for the first time won't get any message at all. I have a plan on how to improve this and I'll keep this issue in mind. |
i thought people read projects' README before they install, so if they |
Haha just check out issues ;) |
i know, but i bet its not even 2% of the consumers |
it doesnt make sense to install in CI, so it would be nice to skip installation in ci, like husky does
https://travis-ci.org/nordnet/grid/jobs/241978766#L176
The text was updated successfully, but these errors were encountered: