Register file-pr and split-stack in the README, add a PR template - #45
Merged
Conversation
PR #43 shipped both skills without adding them to either registry. Adds a pre-merge checklist item so the next one doesn't. Also repoints the stale testing-patterns row at tdd, which replaced it in #40 — skills/testing-patterns has not existed since. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Registers the newly added workflow skills on the repo’s front page and adds a pull request template to prevent future skills from being merged without being listed in README.md / AGENTS.md.
Changes:
- Update
README.mdto replace the deadtesting-patternsentry withtdd, and addfile-pr+split-stackto the skills table (plus a “File PR + Split Stack” pairing note). - Update
AGENTS.mdto includefile-prandsplit-stackin the skills catalog. - Add a GitHub PR template that includes a Pre-merge checklist item to update
README.mdandAGENTS.md.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| README.md | Updates the skill registry (adds file-pr/split-stack, repoints testing-patterns → tdd) and adds a pairing note. |
| AGENTS.md | Adds file-pr and split-stack entries to keep the repo-wide skill catalog in sync. |
| .github/pull_request_template.md | Introduces a PR template with a pre-merge checklist to prevent unregistered skills. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
|
||
| <!-- One line per change, every box checked. Five lines at most. Say what the change does, not how it's built. --> | ||
|
|
||
| - [x] |
justwiebe
approved these changes
Aug 13, 2026
This was referenced Aug 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
file-prandsplit-stackmerged in #43 without landing inREADME.mdorAGENTS.md, so neither skill is discoverable from the front page. This registers both and adds a pre-merge checklist item to a new PR template, so the next skill doesn't slip through the same gap.The README also still pointed at
testing-patterns, whichtddreplaced in #40 —skills/testing-patternshasn't existed since, so that row was a dead link. Repointed it.One thing worth a decision:
file-prwrites the body with--body-file, which replaces the template wholesale. Agent-opened PRs won't carry the checklist unless the skill's own format grows a Pre-merge heading too. #44 is already editingfile-pr, so I left it alone rather than collide.What Changed
file-prandsplit-stackin the README table and AGENTS.mdtesting-patternsrow attddScreenshots
N/A — no UI changes
Pre-merge
README.mdandAGENTS.md?