Skip to content

Commit 5cdc9fb

Browse files
committed
Merge branch 'jn/merge-no-edit-fix'
* jn/merge-no-edit-fix: merge: do not launch an editor on "--no-edit $tag" Conflicts: builtin/merge.c
2 parents b6b3b6a + 3adab6f commit 5cdc9fb

File tree

2 files changed

+14
-1
lines changed

2 files changed

+14
-1
lines changed

builtin/merge.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1324,7 +1324,8 @@ int cmd_merge(int argc, const char **argv, const char *prefix)
13241324
merge_remote_util(commit) &&
13251325
merge_remote_util(commit)->obj &&
13261326
merge_remote_util(commit)->obj->type == OBJ_TAG) {
1327-
option_edit = 1;
1327+
if (option_edit < 0)
1328+
option_edit = 1;
13281329
allow_fast_forward = 0;
13291330
}
13301331
}

t/t7600-merge.sh

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -683,4 +683,16 @@ test_expect_success GPG 'merge --ff-only tag' '
683683
test_cmp actual expect
684684
'
685685

686+
test_expect_success GPG 'merge --no-edit tag should skip editor' '
687+
git reset --hard c0 &&
688+
git commit --allow-empty -m "A newer commit" &&
689+
git tag -f -s -m "A newer commit" signed &&
690+
git reset --hard c0 &&
691+
692+
EDITOR=false git merge --no-edit signed &&
693+
git rev-parse signed^0 >expect &&
694+
git rev-parse HEAD^2 >actual &&
695+
test_cmp actual expect
696+
'
697+
686698
test_done

0 commit comments

Comments
 (0)