docs: point create-prs-with-gh at the writing guide's PR section #31
Loading…
Reference in a new issue
No description provided.
Delete branch "docs/point-gh-skill-at-writing-guide"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Finishes the fold in #29 by pointing
create-prs-with-ghat thehuman-facing-writing-stylePull requests section, closing the routing gap #29's reviews flagged four times.Approach review: The approach is right and needs no alternative.
The change adds one routing sentence to
skills/create-prs-with-gh/SKILL.md, delegating content guidance tohuman-facing-writing-stylewhile the skill keeps its own narrow scope (shell quoting forghbody flags). Three things make this the correct shape:skills/verify-readme/SKILL.md:10already reads "This skill covers what belongs in a README ... invoke thehuman-facing-writing-styleskill", and the new line follows the same scope-then-delegate pattern.human-facing-writing-stylehas a## Pull requestssection covering titles, body, and what gets cut, so the pointer resolves to real content rather than a dangling reference.create-prs-with-gh— would duplicate a section that already has one owner and would drift. A prose pointer keeps a single source of truth, which matters more here because both skills are delivered independently per run.Placement directly under the Goal line, before
# Reference, means the delegation is read before the mechanics, which is the order a reader needs it in. No maintenance or scaling concern.Approach review by Claude Code Opus (opus)
Summary: No actionable issues found.
Single-sentence docs addition to
skills/create-prs-with-gh/SKILL.md. Verified against the working tree:skills/human-facing-writing-style/SKILL.md, and its## Pull requestssection (line 267) covers exactly what the pointer promises — title shapes under### Titlesand body structure under### Body.skills/verify-readme/SKILL.md:10("invoke thehuman-facing-writing-styleskill"), keeping the delegation idiom consistent across the estate.No frontmatter, layout, or
axskillsdelivery contract is touched, so nothing in the source'sskills/<name>/SKILL.mdstructure is affected.Code review by Claude Code Opus (opus)
Closing unmerged: the estate triage decided to delete
create-prs-with-ghrather than point it at the writing guide. Superseded by #32.Pull request closed