# amend param not available to the pre-commit hook

2 messages from 2020-08-02 to 2020-08-02. Participants: Andrea Wyss, Phillip Wood.
Thread: https://gitlist.dev/t/53964

## Andrea Wyss, 2020-08-02 13:22

Subject: amend param not available to the pre-commit hook
Message-ID: <30DFCBF3-FEA8-4FF5-94ED-A81AB10B04A3@gmail.com>
URL: https://gitlist.dev/e/30DFCBF3-FEA8-4FF5-94ED-A81AB10B04A3%40gmail.com

```
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

```

## Phillip Wood, 2020-08-02 15:10

Subject: Re: amend param not available to the pre-commit hook
Message-ID: <f9016e7a-0769-6a55-ab32-6b54674ae2c1@gmail.com>
URL: https://gitlist.dev/e/f9016e7a-0769-6a55-ab32-6b54674ae2c1%40gmail.com
In-Reply-To: <30DFCBF3-FEA8-4FF5-94ED-A81AB10B04A3@gmail.com>

```
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
> 

```