Skip to content
On this page

Python群规委员会工作流程

PR 审查

  1. 委员会成员在 GitHub 或类似的代码托管平台上分配给自己要审查的 PR。
  2. 仔细审查代码、文档和更改的内容,确保其质量、一致性和合理性。
  3. 如果对于提交的 PR 存在任何问题、改进建议或需要进一步讨论的地方,可以在评论中提出 Change Request(变更请求)并添加标签 Needs-Author-Feedback
  4. 如果 PR 的内容符合规范,并且没有争议或不合理之处,委员会成员可以批准该 PR,并添加标签 Member-Approved(成员批准)。
  5. 需要至少有两个委员会成员(或管理员)批准该 PR,确保有足够的审批意见。
  6. 如果存在需要作者进行修改的情况,审查者将在评论中指明,并添加标签 Needs-ModifyNeeds-Author-Feedback,然后移除 Member-Approved 标签(如果有的话)。
  7. 成员需按照实际情况根据标签说明添加标签

PR 合并

  1. 一个 PR 被审查并获得至少两个委员会成员(或管理员)批准,并且不再有任何 Needs-Modify 标签时,它就可以被认为是准备好被合并的。
  2. 在合并之前,确保没有 Blocking-issueDo-Not-Merge 标签存在于该 PR 上,这样可以避免可能存在的问题。
  3. 可以允许自动合并操作,确保代码能够顺利地合并到主代码库中。
  4. 合并完成后,可以添加标签 Policy-Accepted(如果 PR 影响了规则)、Commissions(如果与委员会相关)、Project-File(如果更改了项目文件)等来表示 PR 的状态。

标签说明

  • Administrator-Approved:此 PR/issue 已被委员会管理员批准。
  • Announcement:委员会公告。
  • Blocking-issue:这个 PR/issue 由于已知问题被阻止。
  • Breaking-Change:此 PR/issue 已对规则进行重大更改。
  • Commissions:与委员会相关的 PR/issue。
  • Do-Not-Merge:此 PR/issue 被委员会阻止。
  • Member-Adding:此 PR 会添加委员会成员。
  • Member-Approved:此 PR/issue 已被委员会成员批准。
  • Merge-Needed:此 PR 需要手动合并。
  • Needs-Author-Feedback:此 PR/issue 需要作者反馈。
  • Needs-Modify:此 PR/issue 需要作者修改。
  • Needs-Triage:此 PR/issue 需要进行分类。
  • Policy-Accepted:此 PR/issue 已被委员会接受。
  • Policy-Modify:此 PR/issue 会更改已知规则。
  • Policy-Rejected:此 PR/issue 已被委员会拒绝。
  • Policy-Remove:此 PR/issue 会移除已知规则。
  • Policy-Request:此 PR/issue 会添加新的规则。
  • Project-File:此 PR/issue 会更改项目文件。
  • Review-Needed:此 PR/issue 需要被委员会审查。