技术小黑屋

Merge(Pull) Request 推荐的标签列表

一个Merge Request 的 阶段

  1. 代码添加或修改,需要进行review
  2. 代码review结束,需要修改
  3. 重复步骤1和步骤2,直到达到可以合并的标准

角色

  • MR submitter 负责提交Merge Request,并针对review做修改
  • MR reviewer 负责review Merge Request,提出MR中存在的问题,该角色可以对应多个人
  • MR dispatcher 负责分发MR,修改或增加MR reviewer
  • MR terminator 最终负责MR结果走向的人,比如合并或者关闭

注意

  • 上述角色至少需要两个人
  • 因权限问题, MR reviewer 可能无权限合并该MR

有哪些标签

MR:Needs Review(MR:需要Review)

  • 当MR创建或者进行了更新,需要人员Review时,MR submitter 设置该标签
  • 如果MR对应的内容不需要跟版,不需要现在合并的,不要增加该Lable
  • 如果一个MR,当前的label不包含MR:Needs Review,MR reviewer 则不会review

MR:Reviewed With Comments(MR:需要修改)

  • MR reviewer 进行了review,并提出了一些评论来记录发现的问题和疑问
  • MR reviewer 移除 MR:Needs Review 并添加标签 MR:Reviewed With Comments
  • MR submitter 根据提出的问题和疑问进行修改或回答,当修改完毕后,移除标签 MR:Reviewed With Comments,并设置MR:Needs Review

Good to Merge(可以合并)

  • 经过上面的来回操作,在某一点,MR达到了一个可以合并的时候,这时候需要移除前面的标签,设置成Good to Merge
  • 设置这个标签,需要由MR reviewer 操作,而不是MR submitter
  • 设置完这个标签后,MR不需要再更新
  • 因MR reviewer不一定有merge权限,这一标签还是有必要的

Do Not Merge(请勿合并)

  • 禁止合并标签,该MR可以被 review,但是不要合并进来.
  • 适用于未来的需求,目前尚不需要加入到主分支
  • 辅助标签,更加明确表明不需要合并当前MR

待验收

  • 该功能没有开始进行验收(测试,UI,UE,产品,数据等)
  • 如果当前MR包含待验收,通常不会被合并

验收中

  • 该功能正在处于验收中(测试,UI,UE,产品,数据等)
  • 如果当前MR包含该标签,通常不会被合并

验收通过

  • 该功能已经通过验收(测试,UI,UE,产品,数据等)
  • 如果当前MR包含验收通过,可以进行合并

BugFix

  • 仅用于修复线上版本的崩溃提交时,使用

可能不上线

  • 有些MR已经完成,但不确定什么版本引入,需要增加该标签

技术需求





快来解锁最新版 Typora,新用户券后仅需 84 元!
如何便宜的购买 office 365 ?
新版赤友 NTFS 助手来袭,正版超低价