这是一个创建于 1068 天前的主题，其中的信息可能已经有所发展或是发生改变。
RT.
期望不论谁提交的代码，至少需要其他人打分之后才可以通过
首先团队里面目前是认同 CR, 但是 gitlab 没有办法做强制 review, 也没查到很好的方案。
 |
1
dilu 2021-12-10 16:39:32 +08:00
没用的，时间一久就会流于形式
|
 |
2
ebingtel 2021-12-10 16:49:46 +08:00
设置保护分支 没有 review ，就不能合并啊？
|
 |
3
amundsen 2021-12-10 16:56:04 +08:00  1
可以设置机器人的，我公司就是这样做的
|
 |
4
cnoder 2021-12-10 16:59:55 +08:00
不让 merge ，必须提 merge request 给别人呗
|
 |
5
66beta 2021-12-10 17:00:41 +08:00
收回 merge 权限啊
|
 |
6
zhangyichent 2021-12-10 17:05:15 +08:00
我以前公司是有一个工具的，就是安装一下，你提交代码之前就必须从这个工具走 git ，他会发邮件给你指定的几个人。。。不过忘了叫啥了
|
 |
7
beryl OP |
 |
8
SoloCompany 2021-12-10 19:08:05 +08:00 via iPhone
新版的支持 approve 啊，可以收回 merge 权限，分配 reviewer ，要求必须经过 approval 才允许 merge ，负责 merge 的人不需要负责 review
|
 |
9
clf 2021-12-10 19:14:50 +08:00 via Android
@beryl 可以取消 mr 权限，弄个机器人账号有权限，机器人脚本监听到非提交 mr 本人 review 后自动合并。
|
|
10
beryl OP @SoloCompany 这个需要企业付费版本
|
 |
11
qianlifeng 2021-12-10 21:33:45 +08:00
|
 |
12
Leviathann 2021-12-10 21:47:58 +08:00
只给测试和 tech lead merge 权限
|
|
13
SoloCompany 2021-12-11 01:37:56 +08:00 via iPad
@beryl 我们用的是 13 ，好像并不是 ee 专有的功能
|
|
14
beryl OP @SoloCompany 表达错误了，现在这种模式我们也是有的，但是有个问题是，如果把 merge 权限收回来，团队里面可能只有两个人有权限，这样这两个人可能会成为瓶颈，期望的是，自己提交的至少非本人来 review 即可以。
|
 |
15
horizon 2021-12-11 10:35:21 +08:00 via Android
写个 chrome 插件或者注入脚本，自己的 mr 不让点 merge
|
|
17
zhangyichent 2021-12-11 15:13:36 +08:00
gerrit 对了，叫这个。
|
|
18
SoloCompany 2021-12-11 18:29:37 +08:00
@beryl 你还是没理解, 关键在于 review 权限和 merge 权限分开, 既然你不需要控制 merge 只希望控制 review, 而又懒的找个人去点按钮, 甚至可以写个 robot 在 reviewer 点 approve 的同时自动 merge
当然, 如果开发者完全不配合, 自己点 approve 按钮, 而又不想分配人员来行使复核并 merge 的责任, 那么你可以让 robot 只是简单的检查 approver 必须和 submitter 不同, 也就是说 submitter 必须找个人帮他点 approve 按钮 |
|
19
horizon 2021-12-12 17:32:45 +08:00 via Android
@zhangyichent 试试 nginx 注入 js 脚本
|
Select Language