amend param not available to the pre-commit hook

2 messages, 2 authors, 2020-08-02 · open the first message on its own page

amend param not available to the pre-commit hook

From: Andrea Wyss <hidden>
Date: 2020-08-02 13:22:53

Issue
amend param not available to the pre-commit hook

Problem
Cannot write a pre-commit hook that stops amending a commit that is present in other branches.

Steps to Reproduce (git version 2.21.1)
1. Code in the pre-commit hook:
  echo "DEBUG 0='$0' 1='$1'"

2. run:
  git commit --amend

Current Result
  DEBUG 0='.git/hooks/pre-commit' 1=''

Expected Result
  DEBUG 0='.git/hooks/pre-commit' 1='--amend'

Please, let me know if you need other info.
I'm looking forward for a fix or workaround.

Thank you,
Andrea Wyss

Re: amend param not available to the pre-commit hook

From: Phillip Wood <hidden>
Date: 2020-08-02 15:10:58

Hi Andrea

On 02/08/2020 14:22, Andrea Wyss wrote:
Issue
amend param not available to the pre-commit hook

Problem
Cannot write a pre-commit hook that stops amending a commit that is present in other branches.

Steps to Reproduce (git version 2.21.1)
1. Code in the pre-commit hook:
   echo "DEBUG 0='$0' 1='$1'"

2. run:
   git commit --amend

Current Result
   DEBUG 0='.git/hooks/pre-commit' 1=''

Expected Result
   DEBUG 0='.git/hooks/pre-commit' 1='--amend'
I'm not sure why you expect this, the documentation for the pre-commit hook states 'this hook takes no parameters'

That does not mean that it would not be useful to have this information available to the pre-commit and possibly prepare-commit-msg hooks but it is not a bug. I'm not sure that adding a parameter to the hook is a good route as it may break existing hooks that do not expect any parameters, maybe we could pass it in the environment.

Best Wishes

Phillip
Please, let me know if you need other info.
I'm looking forward for a fix or workaround.

Thank you,
Andrea Wyss
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help