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

update npm missing script matchers #1367

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

songz
Copy link

@songz songz commented Mar 28, 2023

Closes #1320

Npm error logs has changed:

Screenshot 2023-03-28 at 9 33 35 AM

Without this change, fuck will give either an irrelevant script or no fucks given message.

With this change, an appropriate suggestion will be given:
Screenshot 2023-03-28 at 9 34 57 AM

@BuZZ-T
Copy link

BuZZ-T commented Sep 14, 2023

It's lowercase for me:

npm ERR! missing script: lit
npm ERR! 
npm ERR! Did you mean this?
npm ERR!     lint

(Ubuntu 20.04)
If it's different for other environments, both lower- and uppercase version should be allowed

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.

Feature: npm missing script, did you mean this
2 participants