Skip to content

docs(pr): 统一贡献与 Ready 操作说明 - #166

Merged
axiomoth merged 2 commits into
mainfrom
docs/pr-ready-guidance-20261013
Oct 13, 2026
Merged

axiomoth merged 2 commits into
mainfrom
docs/pr-ready-guidance-20261013

Conversation

@splrad-steward

@splrad-steward splrad-steward Bot commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

摘要

更新 PR 提交流程说明,补充 Steward 自动建 PR 机制、手动兜底条件及 Ready for review 操作权限范围。

变更原因

Steward 会自动创建或重建 PR 标题正文,需明确手动兜底触发条件,并厘清作者与协作者对 Ready for review 的操作权限。

主要改动

  • 说明有写权限时推送后等待 Steward 自动创建或更新草稿 PR
  • 列出机器人故障、无法自动创建且仍无 PR 时才手动补建的判断顺序
  • 补充 fork 贡献者创建 PR 时无需等待机器人代开
  • 明确 PR 作者及拥有目标仓库 Write 及以上权限者均可点击 Ready for review
  • 说明 Ready 操作适用于同仓和 fork PR,且 fork 作者无需上游 Write 权限
  • 指出 Ready 仅表示提交审查,检查、批准和合并仍需分别完成
  • 将验证命令、版本和未覆盖环境记录位置由人工补充章节改为 PR 评论
  • 提醒 Steward 可能重建 PR 标题正文,补充说明应写入 PR 评论

影响分析

  • 贡献者需按新流程等待或判断是否需手动创建 PR
  • 手动创建 PR 时不再将补充证据写入人工补充章节,而改写入 PR 评论

贡献者:@axiomoth

Signed-off-by: Axiomoth <alearner@splrad.com>
Copilot AI balanced review requested due to automatic review settings October 13, 2026 14:38
@splrad-steward splrad-steward Bot added the chore 其他仓库维护、元数据或一般配置。 label Oct 13, 2026
@splrad-steward splrad-steward Bot added refactor 重组生产代码且保持外部行为不变。 and removed chore 其他仓库维护、元数据或一般配置。 labels Oct 13, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

文档改动与仓库现有贡献规则一致,未发现待解决问题。

0 open findings

What changed in this PR

统一 CONTRIBUTING.md 中 PR 创建、草稿转 Ready 及验证证据的协作流程。

Changes:

  • 明确 Steward 自动创建或更新 PR 的流程。
  • 区分同仓与 fork PR 的 Ready 操作权限。
  • 说明验证证据应通过 PR 评论保留。
File Description
CONTRIBUTING.md 更新 PR 创建、Ready 权限及验证记录说明。

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@axiomoth
axiomoth marked this pull request as ready for review October 13, 2026 15:17
@axiomoth
axiomoth requested a review from a team October 13, 2026 15:17
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-13T16:09:07.064594Z fdafdbb New commits
🔒 Security Review ✅ Completed 2026-10-13T16:08:50.079737Z fdafdbb New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Signed-off-by: Axiomoth <alearner@splrad.com>
Copilot AI balanced review requested due to automatic review settings October 13, 2026 16:06
@splrad-steward splrad-steward Bot added chore 其他仓库维护、元数据或一般配置。 and removed refactor 重组生产代码且保持外部行为不变。 labels Oct 13, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

文档变更与仓库现有贡献流程一致,未发现待解决问题。

0 open findings

🧠 Review effort: Balanced

@axiomoth
axiomoth merged commit f4e3fd6 into main Oct 13, 2026
9 checks passed
@axiomoth
axiomoth deleted the docs/pr-ready-guidance-20261013 branch October 13, 2026 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore 其他仓库维护、元数据或一般配置。

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants