Skip to content

Add deprecation notice for affected chapters#930

Open
softchris wants to merge 2 commits into
mainfrom
softchris-patch-1
Open

Add deprecation notice for affected chapters#930
softchris wants to merge 2 commits into
mainfrom
softchris-patch-1

Conversation

@softchris

Copy link
Copy Markdown
Contributor

Added deprecation notice for Sampling in README.

Added deprecation notice for Sampling in README.
Copilot AI review requested due to automatic review settings July 9, 2026 19:41
@softchris softchris marked this pull request as draft July 9, 2026 19:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a prominent deprecation callout to the “Sampling” lesson to reflect that the MCP specification release candidate deprecates Sampling (and to direct readers to relevant release-candidate context).

Changes:

  • Added a top-of-file “DEPRECATED” banner linking to external release notes for the 2026-07-28 release candidate.
  • Left the existing in-document deprecation blockquote in place (resulting in duplicate deprecation messaging).
Show a summary per file
File Description
03-GettingStarted/14-sampling/README.md Adds an additional deprecation notice/link at the top of the Sampling chapter README.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Low

@softchris softchris requested a review from liamchampton July 9, 2026 19:57
@softchris softchris marked this pull request as ready for review July 9, 2026 19:58

@liamchampton liamchampton left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@softchris - for each of the depreciation notices, maybe it should be hyper-linked to the spec RC?

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