Web lists-archives.com

Re: [PATCHv2] builtin/merge: honor commit-msg hook for merges




Stefan Beller <sbeller@xxxxxxxxxx> writes:

>> I also thought that we were hunting calls of cmd_foo() from outside
>> the git.c command dispatcher as grave errors and want to clean up
>> the codebase to get rid of them.
>
> ... but I did not account for this fact. (I was not aware of these being
> called grave errors, but assumed this is a good state. And why change
> a good state?)

https://public-inbox.org/git/20170830053108.g2xsn43rwulnwn3p@xxxxxxxxxxxxxxxxxxxxx/

gives a good explanation why it is not a good state.