Appearance
Python群规委员会工作流程
PR 审查
- 委员会成员在 GitHub 或类似的代码托管平台上分配给自己要审查的 PR。
- 仔细审查代码、文档和更改的内容,确保其质量、一致性和合理性。
- 如果对于提交的 PR 存在任何问题、改进建议或需要进一步讨论的地方,可以在评论中提出 Change Request(变更请求)并添加标签
Needs-Author-Feedback
。 - 如果 PR 的内容符合规范,并且没有争议或不合理之处,委员会成员可以批准该 PR,并添加标签
Member-Approved
(成员批准)。 - 需要至少有两个委员会成员(或管理员)批准该 PR,确保有足够的审批意见。
- 如果存在需要作者进行修改的情况,审查者将在评论中指明,并添加标签
Needs-Modify
和Needs-Author-Feedback
,然后移除Member-Approved
标签(如果有的话)。 - 成员需按照实际情况根据标签说明添加标签
PR 合并
- 一个 PR 被审查并获得至少两个委员会成员(或管理员)批准,并且不再有任何
Needs-Modify
标签时,它就可以被认为是准备好被合并的。 - 在合并之前,确保没有
Blocking-issue
或Do-Not-Merge
标签存在于该 PR 上,这样可以避免可能存在的问题。 - 可以允许自动合并操作,确保代码能够顺利地合并到主代码库中。
- 合并完成后,可以添加标签
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 需要被委员会审查。