Hi Orgad On Tue, 26 Jul 2016, Orgad Shaneh wrote:
> From: Orgad Shaneh <[email protected]> This is unnecessary, as it matches your email address. > include also commit-msg hook. This comment was a bit cryptic, until I read the patch. Now I find that comment redundant with the patch. However, I think that... > - OPT_BOOL('n', "no-verify", &no_verify, N_("bypass pre-commit > hook")), > + OPT_BOOL('n', "no-verify", &no_verify, N_("bypass pre-commit > and commit-msg hooks")), ... it may be more desirable to future-proof this simply by saying "bypass hooks". In the alternative, it would be good if the commit message could convincingly make the case that there are no other hooks that will be skipped with -n. Of course, I could go and look at the source code to convince myself. But it is really the duty of the commit message to be already convincing enough. Ciao, Dscho -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
