Skip to content

feat(prepare-commit-msg-hook): enhance commit message formatting with a divider and instructions for better user guidance #492

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

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

Conversation

PhantasWeng
Copy link

@PhantasWeng PhantasWeng commented Jul 1, 2025

According to the discussion at #490, modify the git commit template during the hook to the default comment stats to align with general git commit practices.

終端機 - phantasweng — owlting -l — owlting — ssh -t ubuntu@192 168 70 92 tmux new -A -s server — 237×60_2025_07_01 at 14 09 35

終端機 - phantasweng — owlting -l — owlting — ssh -t ubuntu@192 168 70 92 tmux new -A -s server — 237×60_2025_07_01 at 14 02 27

… a divider and instructions for better user guidance
@PhantasWeng PhantasWeng marked this pull request as ready for review July 1, 2025 06:10
@muni-corn
Copy link

this is really nice. i'd like to see this feature be configurable, because i like the current behavior as-is, and i can just cancel commits with :cq in helix (and vim).

@PhantasWeng
Copy link
Author

this is really nice. i'd like to see this feature be configurable, because i like the current behavior as-is, and i can just cancel commits with :cq in helix (and vim).

In fact, you can now do this using oco hook set.
This PR mainly aligns the default behavior with the original git behavior.

Do you mean that you think we need to add a config key to choose whether or not to automatically uncomment?

@muni-corn
Copy link

Do you mean that you think we need to add a config key to choose whether or not to automatically uncomment?

yeah, that's what i meant 😁

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