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

fix: reuse commit-message for in-action commit #15

Merged
merged 3 commits into from
Jul 1, 2024

Conversation

yajo
Copy link
Contributor

@yajo yajo commented Jun 30, 2024

If the repo uses some kind of commit message linting, such as a pre-commit hook that implements commitizen, it will be able to still do the commit without linter violations.

@mhils
Copy link
Member

mhils commented Jun 30, 2024

Would --no-verify work for you as well?

If the repo uses some kind of commit message linting, such as a pre-commit hook that implements commitizen, it will be able to still do the commit without linter violations.
@yajo yajo force-pushed the reuse-commit-msg branch from 138a087 to c80dcf0 Compare July 1, 2024 06:29
@yajo
Copy link
Contributor Author

yajo commented Jul 1, 2024

Yes, too.

@mhils
Copy link
Member

mhils commented Jul 1, 2024

In this case I would much prefer to use --no-verify, then it works out of the box and does not require configuration.

@yajo
Copy link
Contributor Author

yajo commented Jul 1, 2024

Fine, changed

Copy link
Member

@mhils mhils left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 🍰

@mhils mhils merged commit 2891949 into autofix-ci:main Jul 1, 2024
1 check passed
@yajo yajo deleted the reuse-commit-msg branch July 2, 2024 17:06
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.

2 participants