libgit2 / libgit2/libgit2sharp

NormalMerge should call callback on conflicts

未关闭
#1,874 0 条评论 1 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

主要语言
C#
星标
3.5k
派生
925
PR 合并指标
30 天内没有已合并 PR

描述

Reproduction steps
        [Fact]
        public void NormalMergeShouldCallCallbackOnConflicts()
        {
            string committishToMerge = "fast_forward";

            using (var repo = new Repository(SandboxMergeTestRepo()))
            {
                Touch(repo.Info.WorkingDirectory, "b.txt", "this is an alternate change");
                var conflicts = 0;
                var options = new MergeOptions
                {
                    OnCheckoutNotify = (path, flags) =>
                {
                    conflicts++;
                    return true;
                },
                CheckoutNotifyFlags = CheckoutNotifyFlags.Conflict,
                FastForwardStrategy = FastForwardStrategy.NoFastForward,
            };

                Assert.Throws<CheckoutConflictException>(() => repo.Merge(committishToMerge, Constants.Signature, options));
                Assert.Equal(1, conflicts) ;
            }
        }

Expected behavior

Test shouldn't fail

Actual behavior

Test fails

Version of LibGit2Sharp (release number or SHA1)

df3b22a754ef56da8d7e3c330ce2d783c2b7982e

Operating system(s) tested; .NET runtime tested

Windows netcore 2.1

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

调研方向

从 NormalMergeShouldCallCallbackOnConflicts 测试以及使用 MergeOptions.OnCheckoutNotify、CheckoutNotifyFlags.Conflict 和 NoFastForward 的 repo.Merge 调用开始。重现该失败,跟踪冲突处理,并确认 callback 执行一次,同时仍会引发 CheckoutConflictException。

由索引模型根据 Issue 内容生成。

评估

技术栈
csharp, git
领域
backend, testing
Issue 类型
缺陷
难度
3/5
预计耗时
1-2 天
活跃度
停滞
描述清晰度
描述清楚
新手友好度
45/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。