PR 协调员
本节介绍 PR 协调员的职责。有关如何进行良好审查的更多信息,请参阅 审查变更。
职责
每周轮班担任 PR 协调员的每天
- 审查 打开的拉取请求,检查其质量是否符合 样式 和 内容 指南。
- 从最小的 PR(
size/XS
)开始,最后是最大的(size/XXL
)。尽可能多地审查 PR。
- 从最小的 PR(
- 确保 PR 贡献者签署了 CLA。
- 使用 此 脚本提醒未签署 CLA 的贡献者签署。
- 提供变更反馈,并向其他 SIG 的成员请求技术审查。
- 为建议的内容变更在 PR 中提供内联建议。
- 如果您需要验证内容,请在 PR 中发表评论并请求更多详细信息。
- 分配相关的
sig/
标签。 - 如果需要,从文件的前端数据中的
reviewers:
块中分配审阅者。 - 您也可以通过在 PR 上评论
@kubernetes/<sig>-pr-reviews
来标记 SIG 以进行审查。
- 使用
/approve
评论批准 PR 以合并。准备好后合并 PR。 - 还要检查针对 参考文档生成器 代码的拉取请求,并审查这些请求(或寻求帮助)。
- 支持 问题协调员 每天对传入问题进行分类和标记。请参阅 分类和标记问题,了解 SIG Docs 如何使用元数据的指南。
注意
PR 协调员的职责不适用于本地化 PR(非英语 PR)。本地化团队有自己的流程和团队来审查其语言 PR。但是,确保语言 PR 标记正确、审查小的与语言无关的 PR(如链接更新)或在长期 PR(在 6 个月前打开且一个月以上未更新)中标记审查者或贡献者通常很有帮助。对协调员有用的 GitHub 查询
以下查询在协调时很有帮助。在完成这些查询后,剩余的需要审查的 PR 列表通常很小。这些查询排除了本地化 PR。所有查询都是针对主分支的,除了最后一个。
- 没有 CLA,无法合并:提醒贡献者签署 CLA。如果机器人和人都提醒过他们,请关闭 PR 并提醒他们可以在签署 CLA 后重新打开它。 **请勿审查作者未签署 CLA 的 PR!**
- 需要 LGTM:列出需要成员 LGTM 的 PR。如果 PR 需要技术审查,请循环到机器人建议的审阅者之一。如果内容需要修改,请在内联中添加建议和反馈。
- 有 LGTM,需要文档批准:列出需要
/approve
评论才能合并的 PR。 - 快速获胜:列出针对主分支且没有明显阻塞的 PR。(在您完成 PR 时更改大小标签中的“XS”[XS、S、M、L、XL、XXL])。
- 不是针对主要分支:如果 PR 针对
dev-
分支,则它适用于即将发布的版本。使用以下方法分配 文档发布经理:/assign @<manager's_github-username>
。如果 PR 针对旧分支,请帮助作者确定它是否针对最佳分支。
对协调员有用的 Prow 命令
# add English label
/language en
# add squash label to PR if more than one commit
/label tide/merge-method-squash
# retitle a PR via Prow (such as a work-in-progress [WIP] or better detail of PR)
/retitle [WIP] <TITLE>
何时关闭拉取请求
审查和批准是缩短我们的 PR 队列并保持其最新状态的一种工具。另一个工具是关闭。
关闭以下情况的 PR:
作者在两周内未签署 CLA。
作者可以在签署 CLA 后重新打开 PR。这是一种低风险的方式,可以确保没有任何内容未经签署 CLA 就合并。
作者在 2 周或更长时间内未回复评论或反馈。
不要害怕关闭拉取请求。贡献者可以轻松地重新打开并恢复正在进行的工作。通常,关闭通知会促使作者恢复并完成他们的贡献。
要关闭拉取请求,请在 PR 上留下 /close
评论。
注意
k8s-triage-robot
机器人在 90 天不活动后将问题标记为过时。再过 30 天,它会将问题标记为腐烂并关闭它们。PR 协调员应在 14-30 天不活动后关闭问题。PR 协调员影子计划
在 2021 年后期,SIG Docs 推出了 PR 协调员影子计划。该计划的推出是为了帮助新贡献者了解 PR 协调流程。
成为影子
如果您有兴趣作为 PR 协调员的影子,请访问 PR 协调员 Wiki 页面,查看今年的 PR 协调时间表并注册。
Kubernetes 组织成员可以编辑 PR 协调员 Wiki 页面 并注册为现有 PR 协调员的影子,持续一周。
其他人可以在 #sig-docs Slack 频道 上联系请求在特定的一周内成为已分配 PR 协调员的影子。您可以随时联系 Brad Topol(
@bradtopol
)或 SIG Docs 联合主席/领导 之一。注册为 PR 协调员的影子后,请在 Kubernetes Slack 上向 PR 协调员介绍自己。
最后修改时间:2024 年 3 月 19 日 下午 1:03 PST:修复 Markdown (2f55942c7e)