审查拉取请求中的建议更改

在拉取请求中,您可以审查和讨论提交、更改的文件以及基本和比较分支中文件之间的区别(或“差异”)。

关于审查拉取请求

您可以在拉取请求中每次审查一个文件的更改。 在拉取请求中审查文件时,可以对特定更改留下单个注释。 在完成审查每个文件后,您可以将该文件标记为已查看。 这会折叠文件,帮助您识别还需要审查的文件。 拉取请求标题中的进度条显示您查看过的文件数。 在按需要审查文件后, 您可以提交包含摘要评论的审查来批准拉取请求或请求额外更改。

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

开始审查

  1. 在仓库名称下,单击 拉取请求
    ![议题和拉取请求选项卡选择](/assets/images/help/repository/repo-tabs-pull-requests.png)
    
  2. 在拉取请求列表中,单击要审查的拉取请求。
  3. 在拉取请求中,单击 Files changed(文件已更改)

Pull Request Files changed tab

  1. 将鼠标悬停在您要添加评论的代码行上,然后单击蓝色评论图标。 要在多行上添加评论,请单击并拖动以选择行范围,然后单击蓝色评论图标。 蓝色评论图标
  2. 在评论字段中输入您的评论。 评论字段
  3. (可选)要建议对一行或多行进行特定更改,请单击 ,然后在建议块内编辑文本。 建议块
  4. 完成后,单击 Start a review(开始审查)。 如果已开始审查,您可以单击 Add review comment(添加审查注释)开始审查按钮

提交审查之前,您的行注释为待处理状态并且仅对您可见。 您可以在提交审查之前随时编辑待处理的注释。 要取消待处理的审查(包括所有其待处理的注释),请在 Conversation(对话)选项卡中向下滚动到时间表的末尾,然后单击 Cancel review(取消审查)

取消审查按钮

查看依赖项更改

注意:依赖项审查目前处于测试阶段,可能会更改。

如果拉取请求包含对依赖项的更改,您可以使用清单或锁定文件的依赖项审阅来查看更改的内容,并检查更改是否引入安全漏洞。 更多信息请参阅“审查拉取请求中的依赖项更改”。

  1. 在拉取请求中,单击 Files changed(文件已更改)

Pull Request Files changed tab

  1. 在清单或锁定文件标头的右侧,单击 多差异按钮以显示依赖项审查。

    多差异按钮

  2. 您可能还想查看源差异,因为清单或锁定文件可能会发生变化,但不会更改依赖项,也可能存在 GitHub 无法解析的依赖项,因此,这些依赖项不会显示在依赖项审核中。

    要返回到源差异视图,请单击 按钮。

    源差异按钮

将文件标记为已查看

在完成审查文件后,您可以将文件标记为已查看,该文件将会收起。 如果查看过的文件有更改,将会取消已查看的标记。

  1. 在拉取请求中,单击 Files changed(文件已更改)

Pull Request Files changed tab 2. 在完成审查的文件的标头右侧,选择已查看已查看复选框

提交审查

完成审查拉取请求中需要查看的所有文件后,提交您的审查。

  1. 在拉取请求中,单击 Files changed(文件已更改)

Pull Request Files changed tab

  1. 在更改的代码上方,单击 Review changes(审查更改)审查更改按钮
  2. 输入评论,概述您对拟议更改的反馈。 审查摘要评论窗口
  3. 选择您想要留下的审查类型: 具有审查选项的单选按钮
    • 选择 Comment(注释)留下一般反馈而不明确批准更改或请求其他更改。
    • 选择 Approve(批准)提交反馈并批准合并拉取请求中提议的更改。
    • 选择 Request changes(请求更改)提交在拉取请求合并之前必须解决的反馈。
  4. 单击 Submit review(提交审查)

提示

  • 如果启用了必需的审查,并且对仓库具有写入管理员所有者访问权限的协作者提交请求更改的审查,则在同一协作者提交另一次要求批准拉取请求中更改的审查之前,拉请求不能合并。
  • 即使没有获得批准审查,或者请求更改的审查者离开了组织或者联系不上,仓库所有者和管理员也可合并拉取请求。
  • 如果同时启用了必需审查和过期审查,并且代码修改提交已推送到批准的拉取请求分支,则批准将予驳回。 拉取请求必须经过再次审查和批准才可合并。
  • 当多个打开的拉取请求时分别有指向同一提交的头部分支时,如果一个或两者有待定或被拒绝的审查,您将无法合并它们。
  • If your repository requires approving reviews from people with write or admin permissions, then any approvals from people with these permissions are denoted with a green check mark, and approvals from people without these permissions have a gray check mark. Approvals with a gray check mark do not affect whether the pull request can be merged.
  • 拉取请求作者无法批准自己的拉取请求。

延伸阅读

此文档对您有帮助吗?

隐私政策

帮助我们创建出色的文档!

所有 GitHub 文档都是开源的。看到错误或不清楚的内容了吗?提交拉取请求。

做出贡献

或, 了解如何参与。