如何为GitHub私有仓库的Git工作流配置code review流程？

GitHub本身原生内置了完整的分支管控和PR审核能力，完全可以满足你的需求，无需引入额外第三方工具，具体配置流程如下：

1. 配置master分支保护规则 #

进入对应私有仓库后台完成以下配置：

• 点击仓库顶部的 Settings 选项卡
• 在左侧边栏找到 Code and automation 分类下的 Branches 选项
• 点击 Branch protection rule 区域的 Add rule 按钮
• 在 Branch name pattern 输入框填写 master，匹配要管控的主分支
• 勾选以下核心配置项：
  • Require a pull request before merging：强制所有代码合入主分支必须先提交PR，禁止任何人直接推送代码到master
  • Require approvals：将所需审批人数设置为1
  • Require review from Code Owners：提前在仓库根目录创建名为CODEOWNERS的文件，在文件中写入* @你的GitHub用户名，再勾选本选项后，只有你有权限审批通过PR，其他同事的审批不会生效
  • Do not allow bypassing the above settings：禁止包括仓库管理员在内的所有角色绕过上述规则
  • 可选配置：如果需要强制提交前自动跑测试验证，可以勾选Require status checks to pass before merging，提前配置好CI流水线后，只有所有自动化测试用例执行通过，PR才能进入人工审核环节

2. 团队协作流程落地 #

配置完成后，按以下流程执行即可：

• 所有开发人员需要开发功能时，从最新的master分支拉取独立的开发分支进行编码
• 开发完成后提交PR，PR标题需清晰标注变更类型和内容概要
• PR提交后系统会自动触发规则校验，校验通过后通知你进行审核
• 你可以在线查看逐行代码变更，对有问题的位置添加批注，要求开发人员修改后重新提交
• 可选开启Require conversation resolution before merging配置，强制所有你提出的批注问题都被标记为已解决后，才能执行合并操作
• 只有你审批通过后，PR才有权限执行合并到master的操作，合并完成后可以设置自动删除源分支，减少仓库冗余分支

内容的提问来源于stack exchange，提问作者Hemant Bhargava