PR 管理者

    本节介绍 PR 管理者的职责。关于如何提供较好的评审意见,可参阅 .

    在为期一周的轮值期内,PR 管理者要:

    • 每天对新增的 Issues 判定和打标签。参见 对 Issues 进行判定和分类 以了解 SIG Docs 如何使用元数据的详细信息。

    • 检查 的质量并确保它们符合 样式指南和 要求。

      • 首先查看最小的 PR(),然后逐渐扩展到最大的 PR(size/XXL),尽可能多地评审 PR。
    • 确保贡献者完成 CLA 签署。

    • 针对提供提供反馈,请求其他 SIG 的成员进行技术审核。

      • 为 PR 所建议的内容更改提供就地反馈。
      • 如果您需要验证内容,请在 PR 上发表评论并要求贡献者提供更多细节。
      • 设置相关的 sig/ 标签。
      • 如果需要,从文件开头的 reviewers: 块中指派评阅人。
      • PR 在被合并之前,应该有来自其他成员的 /lgtm 评论。

    执行管理操作时,以下查询很有用。完成以下这些查询后,剩余的要审阅的 PR 列表通常很小。 这些查询都不包含本地化的 PR,并仅包含主分支上的 PR(除了最后一个查询)。

    • : 提醒贡献者签署 CLA。如果机器人和审阅者都已经提醒他们,请关闭 PR,并提醒他们在签署 CLA 后可以重新提交。 在作者没有签署 CLA 之前,不要审阅他们的 PR!

    • 需要 LGTM: 列举需要来自成员的 LGTM 评论的 PR。 如果需要技术审查,请告知机器人所建议的审阅者。 如果 PR 继续改进,就地提供更改建议或反馈。

    • : 列举需要 /approve 评论来合并的 PR。

    • 快速批阅: 列举针对主分支的、没有明确合并障碍的 PR。 在浏览 PR 时,可以将 “XS” 尺寸标签更改为 “S”、”M”、”L”、”XL”、”XXL”。

    • : 如果 PR 针对 分支,则表示它适用于即将发布的版本。 请添加带有 /assign @<负责人的 github 账号>,将其指派给 发行版本负责人。 如果 PR 是针对旧分支,请帮助 PR 作者确定是否所针对的是最合适的分支。

    何时关闭 PR

    当以下条件满足时,可以关闭 PR:

    • 作者两周内未签署 CLA。 PR 作者可以在签署 CLA 后重新打开 PR,因此这是确保未签署 CLA 的 PR 不会被合并的一种风险较低的方法。

    • 作者在两周或更长时间内未回复评论或反馈。

    不要害怕关闭 PR。贡献者可以轻松地重新打开并继续工作。 通常,关闭通知会激励作者继续完成其贡献。

    要关闭 PR,请在 PR 上输入 /close 评论。