12

How to prevent git from pushing commits that contain given string in commit massage, e.g. "DO NOT PUSH" ?

Context/usecase:

My typical workflow is: I hack hack, splitting work into micro commits, once things work I rewrite history, changing order of commit to group them reasonably and later squashing into bigger meaningful pieces. Once work is ready, things are ready to push!

Now I would like git to prevent me from accidentally pushing into repository commits that are still in progress. I considered keeping "DO NOT PUSH" as part of commit message. How to make git automatically prevent me from pushing when it reaches such commit after git push?

(On for pre-receive hook solutions: let's consider github as example service, which AFAIK does not allow pre-recevie hooks, except in its "Enterprise" edition)

Community
  • 1
  • 1
Grzegorz Wierzowiecki
  • 10,545
  • 9
  • 50
  • 88

1 Answers1

10

You can use a pre-push hook, but that remains a local hook which can be bypassed.
See this pre-push example which does look at each commit message

# Check for foo commit
        commit=`git rev-list -n 1 --grep '^foo' "$range"`
        if [ -n "$commit" ]
        then
echo "ERROR: git pre-push hook found commit message starting with 'foo' in $local_ref"

But the best way remains a pre-receive hook on the server side though. That way, the policy is enforced for all contributors. This is not always possible (when you don't have direct access to the remote server like GitHub, BitBucket or GitLab.com)

VonC
  • 1,262,500
  • 529
  • 4,410
  • 5,250
  • @Leon Sure, added. – VonC Feb 05 '17 at 08:25
  • As mentioned above, this is the correct answer but the validation must be on the server side as well. Any developer can disable the local hooks so you must verify it on the server. The only thing that you need to verify is that the answer use `-n 1` which will only check the last commit so you need to do it on all of your commits since user may push several commits at once. – CodeWizard Feb 05 '17 at 09:56