Skip to content

chore: add cross-repo checklist to PR template and teach ship-pr to use it#125

Merged
fullstackjam merged 1 commit into
mainfrom
chore/pr-template-cross-repo-checklist
Jun 2, 2026
Merged

chore: add cross-repo checklist to PR template and teach ship-pr to use it#125
fullstackjam merged 1 commit into
mainfrom
chore/pr-template-cross-repo-checklist

Conversation

@fullstackjam
Copy link
Copy Markdown
Collaborator

What does this PR do?

Adds a cross-repo checklist section to the PR template and updates the ship-pr skill to read the template before opening a PR instead of using a hardcoded body.

Why?

When a change spans multiple repos (e.g. CLI feature + docs update in openboot.dev), it's easy to forget the other repo. A checklist in the PR template surfaces the question at the right moment — for both humans and agents.

The ship-pr skill was previously writing its own --body unconditionally, which would silently discard the template. It now reads .github/pull_request_template.md first and fills in its sections.

Testing

  • go vet ./... passes
  • Relevant tests added or updated
  • Tested locally (./openboot install --dry-run or similar)

Cross-repo checklist

  • Does this need a docs/content update in openboot.dev?
  • Does this change the CLI ↔ server API contract? (update openboot-contract first)

Notes for reviewer

Skill files under .claude/skills/ are not Go code — no unit tests apply. The change is verified by reading the before/after of the two modified files.

@fullstackjam fullstackjam merged commit 7d59ddd into main Jun 2, 2026
12 checks passed
@fullstackjam fullstackjam deleted the chore/pr-template-cross-repo-checklist branch June 2, 2026 15:53
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