目标
通过设置git.code.tencent.com中的项目，达到团队内进行code review的目标
操作流程
- 维护两个分支master分支、dev分支，这两个分支只允许merge request。master分支存放线上稳定版；dev分支存放开发中的版本，用来生成预发布版本或体验版本。
- 开发新功能或多人协作时，从dev分支拉feat-xxx分支或feat-v1-alice分支。（分支名供参考）
- 在feat分支上开发代码，直到该feature完成
- 及时拉取最新的dev分支的代码到本地合并(git pull origin dev –rebase)，防止在merge request的时候冲突
- 登录工蜂系统创建merge request。注意填写Reviewer和Assign to（见“merge request流程”）
- 等待code review通过
- Assign to负责完成合并
- 完成
merge request流程
- 进入项目，点击左边栏“Merge Request”
- 点击“New Merge Request”，选择要merge的分支，点击“Compare branches”。左边是自己的feat分支，右边是dev分支。
- 填写Reviewer和Assign to后提交
code review原则
- 尊重他人，就事论事，对事不对人；
- MR 需从一个需求分支（分支的名字尽量能表达代码的功能）发往上游的 dev 分支；
- MR 中的每一个 commit log 都应该可以和代码对应，方便 review；
- 尽量不要发太大的 MR，以免引起 reviewer 的恐慌；
- 建议保证一个 MR 的粒度和专注，最好不要出现一个 MR 里又有重构又加新 feature 的情况，同样容易引起 reviewer 的恐慌；
- 提 MR 之前请确保在本地或测试环境一切正常；
- reviewee 如果接受 reviewer 提出的修改意见，需要在修改提交以后知晓 reviewer，常见的做法可以是在 review comment 处回复，并带 commit 链接；
- MR 合并者与提交者不能是同一个人