Skip to content

Clean up /pr-merge-ready skill - #7

Merged
a-churchill merged 2 commits into
mainfrom
andrew/pr-merge-ready-weave-cleanup
Oct 1, 2026
Merged

a-churchill merged 2 commits into
mainfrom
andrew/pr-merge-ready-weave-cleanup

Conversation

@a-churchill

Copy link
Copy Markdown
Contributor

Summary

  • Remove guidance tied to the source repository and replace it with repository-appropriate instructions.
  • Keep Weave Checks bot and check-criteria guidance used by this project.
  • Align validation instructions with this repository’s format and test commands.

Verification

  • npm run format
  • npm run test — 405 tests passed

🤖 Generated with Weave Router

Co-Authored-By: Weave Router <router@weaveos.com>
@greptile-apps

greptile-apps Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Retrigger

[Low risk] Documentation updates for a developer workflow skill.

The documented workflow has two non-blocking validation concerns; these findings do not make the PR unsafe to merge.

Reviews (1) · Last reviewed commit: "Clean up /pr-merge-ready skill"

Comment thread .agents/skills/pr-merge-ready/SKILL.md
Comment thread .agents/skills/pr-merge-ready/SKILL.md
Fixed:
- Restore unrelated formatting changes after repository-wide formatting
- Re-run targeted Weave Checks findings before pushing

Co-Authored-By: Weave Router <router@weaveos.com>
@a-churchill
a-churchill merged commit 3422e2e into main Oct 1, 2026
20 checks passed
@a-churchill
a-churchill deleted the andrew/pr-merge-ready-weave-cleanup branch October 1, 2026 00:37
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