The v3 of the patch series addresses the comments from Junio and adds the test to check --amend --signoff does not add signoff if it already exists as suggested by Phillip Wood. Also I have removed two tests from v2 which tested the same thing as other tests in the series, namely: - removed 'commit --include'. This pattern is tested in '--include with staged changes'. - removed 'commit --only'. This pattern is tested in '--only excludes staged changes'. Ghanshyam Thakkar (2): t7501: add tests for --include and --only t7501: add tests for --amend --signoff t/t7501-commit-basic-functionality.sh | 68 ++++++++++++++++++++++++++- 1 file changed, 66 insertions(+), 2 deletions(-) -- 2.43.0