-
Notifications
You must be signed in to change notification settings - Fork 2
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
improvements as pre-commit-hook #15
Conversation
👋 Hello hfudev, we appreciate your contribution to this project! 📘 Please review the project's Contributions Guide for key guidelines on code, documentation, testing, and more. Click to see more instructions ...
Review and merge process you can expect ...
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hfudev Many thanks for your interest in this project and your contribution.
This makes a lot more sense than running a pre-commit hook on every commit or specific file change.
Left a couple of ideas, but overall it LGTM.
22e3206
to
26c2a25
Compare
@tomassebestik thanks for your suggestions. PTAL again! |
HEAD~1
User may now use
to make changelog generated incrementally
HEAD
User may now install the hook to
pre-push
stage, and update the changelog before pushing the branch