-
Notifications
You must be signed in to change notification settings - Fork 15
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
v2.5.1 has a regression #133
Comments
This is happening to me using the |
I am new to mookme (coming from Husky), installed the latest version, followed the docs but ran into the same problem with the pre-commit. Glad its not just me. |
this is what worked for me: rm -rf node_modules
rm -rf ./git/hooks went to then |
My team is seeing the same issue |
Hello ! I didn't realize how breaking this change was... Sorry for the disagreement. My understanding is that npm 10 introduces the breaking change of the "--". I'll need to figure out something for supporting both npm < 10 and npm 10. In the meantime I would suggest applying @alanmynah 's fix or upgrade to npm 10 ? |
I'm on npm version 10.2.3 and still seeing this issue |
Hi, I've noticed that v2.5.1 seems to have a regression that is caused by this commit.
On MacOS Sonoma (14.1.1) I get this error:
The error goes away if I remove the
--
that was added to the command in the pre-commit hook:The text was updated successfully, but these errors were encountered: