我们经常发布文档更新,此页面的翻译可能仍在进行中。有关最新信息,请访问英文文档。如果此页面上的翻译有问题,请告诉我们
文章版本: GitHub.com

关于拉取请求审查

审查允许协作者评论拉取请求中提议的更改、审批更改或在拉取请求合并之前请求进一步更改。 仓库管理员可要求所有拉取请求在合并之前获得批准。

本文内容

关于拉取请求审查

After a pull request is opened, anyone with read access can review and comment on the changes it proposes. You can also suggest specific changes to a line, which the author can apply directly from the pull request. For more information, see "Reviewing proposed changes in a pull request."

仓库所有者和协作者可向具体的个人申请拉取请求审查。 组织成员也可向具有仓库读取权限的团队申请拉取请求审查。 For more information, see "Requesting a pull request review." You can specify a subset of team members to be automatically assigned in the place of the whole team. For more information, see "Managing code review assignment for your team."

审查允许讨论提议的更改,帮助确保更改符合仓库的参与指南及其他质量标准。 您可以在 CODEOWNERS 文件中定义哪些个人或团队拥有代码的特定类型或区域。 当拉取请求修改定义了所有者的代码时,该个人或团队将自动被申请为审查者。 更多信息请参阅“关于代码所有者”。

You can schedule reminders for pull requests that need to be reviewed. For more information, see "Managing scheduled reminders for pull requests."

包含行注释的拉取请求更改的标头

审查有三种可能的状态:

  • 评论:提交一般反馈,但不明确批准更改或申请其他更改。
  • 批准:提交反馈并批准合并拉取请求中提议的更改。
  • 申请更改:提交在拉取请求合并之前必须解决的反馈。

审查状态图像

提示

  • 如果启用了必需的审查,并且对仓库具有写入管理员所有者访问权限的协作者提交请求更改的审查,则在同一协作者提交另一次要求批准拉取请求中更改的审查之前,拉请求不能合并。
  • 即使没有获得批准审查,或者请求更改的审查者离开了组织或者联系不上,仓库所有者和管理员也可合并拉取请求。
  • 如果同时启用了必需审查和过期审查,并且代码修改提交已推送到批准的拉取请求分支,则批准将予驳回。 拉取请求必须经过再次审查和批准才可合并。
  • 当多个打开的拉取请求时分别有指向同一提交的头部分支时,如果一个或两者有待定或被拒绝的审查,您将无法合并它们。

您可以在“对话”时间表中查看拉取请求收到的所有审查,也可在拉取请求的合并框中按仓库所有者和协作者查看审查。

合并框中的审查图像

提示:使用搜索限定符 review-requested:[USERNAME]team-review-requested:[TEAMNAME] 可查找请求您或您所属团队审查的拉取请求。 更多信息请参阅“搜索议题和拉取请求”。

解决对话

如果您打开了拉取请求或者您对被打开拉取请求的仓库具有写入权限,您可以解决拉取请求中的对话。

要指示 Files changed(文件已更改)选项卡上的对话已完成,请单击 Resolve conversation(解决对话)

带解决对话按钮的拉取请求对话

整个对话将被折叠并标记为已解决,以便您更容易找到仍需解决的对话。

已解决对话

If the suggestion in a comment is out of your pull request's scope, you can open a new issue that tracks the feedback and links back to the original comment. 更多信息请参阅“从评论打开议题”。

必要的审查

仓库管理员可以要求所有合并请求在合并到受保护分支之前,获取必要数量的具有仓库写入管理权限的人员或者指定代码所有者的批准。 更多信息请参阅“关于受保护分支”。

启用所需的审查时,具有仓库访问权限的任何人都可以批准拉取请求中的更改。 但若没有必要数量的具有仓库写入管理权限的审查者批准您的拉取请求,您将无法合并拉取请求。 有关仓库权限级别的更多信息,请参阅“组织的仓库权限级别”。如果要求获得指定代码所有者的审查,并且拉取请求会影响具有指定所有者的代码, 则需要该所有者的批准。

更多信息请参阅“关于拉取请求的必需审查”。

Tip: If necessary, people with admin or write access to a repository can dismiss a pull request review. 更多信息请参阅“忽略拉取请求审查”。

延伸阅读

问问别人

找不到要找的内容?

联系我们