Re: hooks not fired by a merge's auto-commit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Richard Bronosky <Richard@xxxxxxxxxxxx> writes:

> Using git 1.7.1 it seems that a merge (more specifically the
> auto-commit) does not fire any of the following hooks:
> pre-commit  prepare-commit-msg commit-msg
>
> Is that by design?

I would probably call it an unintended design that now has long been
established that changing it may break people's existing setups rather
badly.

Especially pre-commit hooks that look for and prevent common mistakes from
happening for individual developers may not want to run, when you are
pulling in other people's work that already has the mess they created.
Such problems are often either too late to fix or you are in no position
to reject their pull requests.  So you would break _my_ workflow (and
others who play similar roles as me, making changes themselves and merging
others work) if you did such a change.

You _could_ add pre-merge to introduce it as a new feature.  For people
who want to apply the same check for merges and individual commits
(e.g. those who never merge other's work), pre-merge can just invoke
pre-commit.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]