-
Notifications
You must be signed in to change notification settings - Fork 914
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
Incorrect Commit Message from Previous Commit #4108
Comments
Just to clarify. This only happens when type-enum is set? |
I can say no since I directly commented out the type-enum rule just to check if it works with other rules but it failed to catch the current commit message. My current workaround is to use commit-msg husky instead of pre-commit hook and it works just fine |
Uhm, ok, but the docs are correct then, right? |
Oh Yeah, it's saying to use commit-msg. This made me think Commitlint does not even handle the pre-commit hook and people shouldn't be using this hook |
👍 In this case this can be closed? |
Yeah, can be closed. But I think this has to be mentioned in the docs since most developers use pre-commit hook to run the Commitlint rules |
You wanna add a hint to the docs? |
Yeah, I'll try to create a PR for this ASAP |
Steps to Reproduce
Current Behavior
No response
Expected Behavior
It should use the current commit message
Affected packages
Possible Solution
No response
Context
No response
commitlint --version
@commitlint/cli
git --version
2.45.2
node --version
20.14.0
The text was updated successfully, but these errors were encountered: