docs: update the README banners - #2940
claude[bot] wants to merge 1 commit into
Conversation
The first banner says what main is, where v1 lives and how long it gets fixes, without the launch-week wording. The second states the pull request limit as it is configured: one open pull request at a time for contributors without write access, drafts not counted.
|
@modelcontextprotocol/client
@modelcontextprotocol/codemod
@modelcontextprotocol/core
@modelcontextprotocol/server
@modelcontextprotocol/server-legacy
@modelcontextprotocol/express
@modelcontextprotocol/fastify
@modelcontextprotocol/hono
@modelcontextprotocol/node
commit: |
There was a problem hiding this comment.
I reviewed this PR and found no bugs; since it is a docs-only banner rewrite with two factual claims the repo cannot confirm, a quick maintainer look is still worthwhile.
What was reviewed:
- Link targets:
.github/ISSUE_TEMPLATE/v2-feedback.yml,docs/migration/upgrade-to-v2.md, andCONTRIBUTING.mdall exist;issues/new/chooseis a standard GitHub path. - The "stable release line" and
v1.xbranch claims matchCONTRIBUTING.mdand the pre-existing banner (server package is at 2.3.0). - The v2 release date (2026-07-27) is new wording and nothing in the repo (changelogs are undated, shallow clone has no tags) confirms it — a maintainer can.
- README.md:12 points to CONTRIBUTING.md right after stating the PR limit, but CONTRIBUTING.md does not mention the limit (it does cover "open an issue first", which is what the sentence is about), so this reads as a soft pointer rather than a contradiction.
Extended reasoning...
The change touches only README.md, rewriting the IMPORTANT and NOTE callout banners at the top; no code, config, or security-sensitive surface is involved. Every link target referenced by the new text resolves to a file or template in the repo, and the branch/stability claims agree with CONTRIBUTING.md. Two points decided defer over approve: the hard release date 2026-07-27 and the "one open PR, drafts excluded" policy are stated as fact but are not verifiable from repository contents, and the file falls under the catch-all CODEOWNERS entry for the typescript-sdk team.
Requested by Felix Weinberger · Slack thread
Before: the README opened with launch-week wording and a
[!WARNING]about pull requests.After: the first banner says what
mainis, where v1 lives and how long it gets fixes, and where to report a problem; the second is a[!NOTE]stating the pull request limit as configured: one open pull request at a time for contributors without write access, drafts not counted.README only, no changeset.
🤖 Generated with Claude Code
https://claude.ai/code/session_012VRbFCp41otcScXE1YY3es
Generated by Claude Code