Skip to content

docs(chapters): 写第五章《PR 与合并权限》并更新路线图 - #14

Merged
jasontopia merged 3 commits into
mainfrom
fm/learn-fm-ch5-v2
Sep 26, 2026
Merged

jasontopia merged 3 commits into
mainfrom
fm/learn-fm-ch5-v2

Conversation

@jasontopia

Copy link
Copy Markdown
Owner

Intent

继续 Learn FirstMate 教程,写第五章。前四章已写完(第一、二章已合入主分支,第三、四章正在走质检流水线)。README 的学习路线图里,第五章的定位是:为什么默认必须船长点头才合并,yolo 姿态放开了什么、没放开什么,以及红色 CI 为什么依然不能合。第五章也是「沿着一件活的真实旅程往下走」这条线(第 2 到 5 章)的最后一章。

What Changed

  • 新增 chapters/05-pr-merge-authority.md:讲清默认必须船长点头才能合并 PR、yolo 姿态只放开合并权限而不放开质检标准、红色 CI 在任何设置下都不可合并(仅 fm-pr-merge.sh --allow-red 可豁免船长当下明确点名的单项检查),并引用本仓库自身 PR 的合并状态作为佐证,附两个只读动手练习。
  • 更新 README.md 的路线图表格,将第 5 章状态从「待写」改为「已写」并链接到新章节文件。

Risk Assessment

✅ Low: Pure documentation addition (new chapter 5 plus a README roadmap link update) with no code, build, or config changes; every behavioral claim traces to AGENTS.md text I verified directly in the local FirstMate checkout, and the PR-status claims about PR #1/#3/#4 match actual GitHub state as of the commit timestamps; the docs check passes.

Testing

This is a documentation-only change (README nav line + new chapters/05-pr-merge-authority.md) in a repo explicitly defined as having no build system, dependency manifest, test framework, or runtime product to drive live. The configured baseline command, scripts/check-docs.py, was re-confirmed passing on the target commit. There is no live-exercisable end-user surface beyond this check, so no further scenarios could be driven against a running product.

  • Live validation: ⚠️ no-surface - 0 of 2 scenarios driven live against the product
Scenario Result Live Evidence
Reader follows README roadmap link into new Chapter 5 ⏸️ untested no This repo is documentation-only with no build system, server, or app to run (per AGENTS.md); there is no live product surface to click through as an end user beyond the automated link/formatting check…
Chapter 5 content accurately reflects FirstMate's merge-authority/YOLO/CI behavior per authoritative source files ⏸️ untested no No executable interface exists to validate a natural-language accuracy claim; this is a content/sourcing review concern (already partly addressed by the prior no-mistakes review commit 34b7369), not s…
  • Outcome: ⚠️ 1 warning across 1 run (27.5s)

Pipeline

Updates from git push no-mistakes

✅ **intent** - passed

✅ No issues found.

🔧 **Rebase** - 1 issue found → auto-fixed ✅
  • ⚠️ README.md - merge conflict rebasing onto origin/main

🔧 Fix applied.
✅ Re-checked - no issues remain.

✅ **Review** - passed

✅ No issues found.

⚠️ **Test** - 1 warning
  • ⚠️ this change has no live-validatable surface; proceed without live validation? (0 of 2 scenarios were driven live against the product); Reader follows README roadmap link into new Chapter 5: This repo is documentation-only with no build system, server, or app to run (per AGENTS.md); there is no live product surface to click through as an end user beyond the automated link/formatting check that already ran as baseline.; Chapter 5 content accurately reflects FirstMate's merge-authority/YOLO/CI behavior per authoritative source files: No executable interface exists to validate a natural-language accuracy claim; this is a content/sourcing review concern (already partly addressed by the prior no-mistakes review commit 34b7369), not something a live product run can confirm or deny.
  • Live validation: ⚠️ no-surface - 0 of 2 scenarios driven live against the product
Scenario Result Live Evidence
Reader follows README roadmap link into new Chapter 5 ⏸️ untested no This repo is documentation-only with no build system, server, or app to run (per AGENTS.md); there is no live product surface to click through as an end user beyond the automated link/formatting check…
Chapter 5 content accurately reflects FirstMate's merge-authority/YOLO/CI behavior per authoritative source files ⏸️ untested no No executable interface exists to validate a natural-language accuracy claim; this is a content/sourcing review concern (already partly addressed by the prior no-mistakes review commit 34b7369), not s…
  • python3 scripts/check-docs.py
  • python3 scripts/check-docs.py (re-run on target commit 34b7369, exit 0, 10 markdown files, no problems found)
✅ **Document** - passed

✅ No issues found.

✅ **Lint** - passed

✅ No issues found.

✅ **Push** - passed

✅ No issues found.

覆盖硬规矩二(未经船长明确同意不合并 PR)、yolo 只管合并权限且不改变
质检流水线本身、以及红色 CI 无论 yolo 或船长指示都不能合并;用第一章
交付时的强推插曲与第二、三章的「合并吧」流程作真实案例。README 路线图
第五行标记为已写并链接新章节,章末预告第六章。
@jasontopia
jasontopia merged commit 9848a29 into main Sep 26, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant