![]() | 1 BrettD 2021-02-27 10:22:55 +08:00 via iPhone ![]() Travis CI 和 GitHub Action 都是自动把 PR 分支 rebase 到 master 之后再开始跑编译和测试,GitLab 不会没有这功能吧 |
![]() | 2 mazyi PRO ![]() |
![]() | 3 learningman 2021-02-27 11:39:06 +08:00 via Android 你这个想法不合理,假设有 n 个分支,那一次 push 就有 2^n 种组合 应该在 PR 的时候检查 |
4 twistedmeadows OP @mazyi 感谢!之前没注意到是这个关键词 |
![]() | 5 sfqtsh 2021-02-27 12:27:56 +08:00 via Android 可强制 fast forward merge |
6 nuistzhou 2021-02-27 15:25:22 +08:00 via iPhone @learningman 楼主说的是 push 当前这个分支,那只需要 rebase 到 master 上然后走 ci 流程就行了啊,关别的分支什么事。 |
7 julyclyde 2021-02-28 09:48:51 +08:00 merge 也是一种 commit 啊。在 commit 的时候测试 |
8 twistedmeadows OP @julyclyde 但是这个时候 merge 已经发生了,如果 fail,还得人工执行手动回退。 我不是说这种代价不可承受,只是觉得 CI 就是用来拦截这种不期望引入的 bug 的。能直接提前发现当然更好。 因为 CI 跑不过的代码连 reviewer 也不必花功夫去审了。 |
9 julyclyde 2021-03-01 12:06:39 +08:00 @twistedmeadows 那你这个需求,可能需要在某个 pre hook 里执行检查吧 |