AndreasAugustin / go-gitmoji-cli

:octocat: A conventional gitmoji commit interactive command line tool for using emojis on conventional commits. :rocket:

Home Page:https://gitmoji.dev/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[Feat]: make it possible to use standard commit message on merge

AndreasAugustin opened this issue · comments

Describe the feature

when the hook is enabled and doing a merge -> The cli is triggered with hook workflow asking for gitmoji and the other conventional gitmoji commit message parts.

Sometimes this is an unwanted behaviour (e.g. on merge). Therefore it should be possible to configure the merge behaviour.

Use Case

doing a merge -> use standard git merge message.

Proposed Solution

make it configurable to use the default git message.

Remark

Also check the hook commit message second message https://git-scm.com/docs/githooks
This might help

prepare-commit-msg
This hook is invoked by [git-commit[1]](https://git-scm.com/docs/git-commit) right after preparing the default log message, and before the editor is started.

It takes one to three parameters. The first is the name of the file that contains the commit log message. The second is the source of the commit message, and can be: message (if a -m or -F option was given); template (if a -t option was given or the configuration option commit.template is set); merge (if the commit is a merge or a .git/MERGE_MSG file exists); squash (if a .git/SQUASH_MSG file exists); or commit, followed by a commit object name (if a -c, -C or --amend option was given).

If the exit status is non-zero, git commit will abort.

The purpose of the hook is to edit the message file in place, and it is not suppressed by the --no-verify option. A non-zero exit means a failure of the hook and aborts the commit. It should not be used as replacement for pre-commit hook.

The sample prepare-commit-msg hook that comes with Git removes the help message found in the commented portion of the commit template.

Acknowledgements

  • I may be able to implement this feature request