Skip to content

Conversation

@domdomegg
Copy link
Member

Adds a GitHub Actions workflow that handles PRs which only modify README.md:

  • Comments explaining we no longer accept new server additions to the README
  • Directs contributors to the MCP registry instead
  • Allows legitimate updates/removals via /i-promise-this-is-not-a-new-server command
  • Uses labels (readme: pending / readme: ready for review) for filtering
  • Minimizes comments after confirmation to reduce clutter

This should reduce the manual effort of closing PRs that are adding new servers to the deprecated list.

Adds a GitHub Actions workflow that:
- Detects PRs that only modify README.md
- Comments explaining we no longer accept new server additions
- Directs contributors to the MCP registry instead
- Allows updates/removals via /i-promise-this-is-not-a-new-server
- Uses labels (readme: pending / readme: ready for review) for filtering
- Minimizes bot comment after confirmation to reduce clutter

Claude-Generated-By: Claude Code (cli/claude-opus-4-5=100%)
Claude-Steers: 10
Claude-Permission-Prompts: 0
Claude-Escapes: 0
@domdomegg
Copy link
Member Author

(I've looked over the implementation, and gone back and forth with Claude a bit with it - I think it's right but hard to tell without actually running it in GitHub Actions so can't be certain)

@domdomegg domdomegg requested a review from olaservo January 5, 2026 18:55
Copy link
Member

@olaservo olaservo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for adding this, lets try it out!

@olaservo olaservo merged commit 4258693 into main Jan 6, 2026
23 checks passed
@olaservo olaservo deleted the adamj/readme-pr-check branch January 6, 2026 04:33
@olaservo
Copy link
Member

olaservo commented Jan 6, 2026

domdomegg added a commit that referenced this pull request Jan 6, 2026
@domdomegg
Copy link
Member Author

Fixed, thanks! New test in #3188

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.

3 participants