Skip to content

chore(deps): update schemars requirement from 0.8 to 1.0 - #258

Merged
4t145 merged 3 commits into
modelcontextprotocol:mainfrom
jokemanfire:deps
Jul 11, 2025
Merged

4t145 merged 3 commits into
modelcontextprotocol:mainfrom
jokemanfire:deps

Conversation

@jokemanfire

Copy link
Copy Markdown
Member

Motivation and Context

How Has This Been Tested?

Breaking Changes

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

@jokemanfire
jokemanfire requested a review from 4t145 June 14, 2025 00:49
@jokemanfire jokemanfire added the T-dependencies Dependencies related changes label Jun 14, 2025
@jokemanfire
jokemanfire marked this pull request as draft June 14, 2025 02:49
@jokemanfire
jokemanfire marked this pull request as ready for review June 14, 2025 03:09
@jokemanfire
jokemanfire marked this pull request as draft June 14, 2025 03:12
@jokemanfire jokemanfire mentioned this pull request Jul 1, 2025
1 of 2 tasks
@github-actions github-actions Bot added T-test Testing related changes T-config Configuration file changes T-core Core library changes T-examples Example code changes T-handler Handler implementation changes labels Jul 6, 2025
@jokemanfire jokemanfire changed the title chore(deps): update schemars requirement from 0.8 to 0.9 chore(deps): update schemars requirement from 0.8 to 1.0 Jul 6, 2025
@jokemanfire
jokemanfire marked this pull request as ready for review July 6, 2025 06:48
@jokemanfire

Copy link
Copy Markdown
Member Author

@4t145 CC

@4t145

4t145 commented Jul 8, 2025 •

Copy link
Copy Markdown
Contributor

@jokemanfire Is there any changes since then? I think you may forgot to push?

@jokemanfire

Copy link
Copy Markdown
Member Author

@jokemanfire Is there any changes since then? I think you may forgot to push?

changed but I rebase it ,because I just resolved the conflict.

@4t145

4t145 commented Jul 8, 2025 •

Copy link
Copy Markdown
Contributor

@jokemanfire Could you resolve the pending review?

@jokemanfire

Copy link
Copy Markdown
Member Author

@jokemanfire Could you resolve the pending review?

Could you point out that you haven't seen any pending reviews o:<

Comment thread crates/rmcp/Cargo.toml Resolved
Comment thread examples/servers/Cargo.toml Resolved
@4t145

4t145 commented Jul 8, 2025

Copy link
Copy Markdown
Contributor

@jokemanfire oh sorry, forget to click the submit button

@jokemanfire
jokemanfire requested a review from 4t145 July 10, 2025 14:05
@4t145
4t145 merged commit 3196c95 into modelcontextprotocol:main Jul 11, 2025
@github-actions github-actions Bot mentioned this pull request Jul 9, 2025
bug-ops pushed a commit to bug-ops/rust-sdk that referenced this pull request Jul 20, 2025
…tprotocol#258)

* chore(deps): update schemars requirement from 0.8 to 0.9

* fix: adapt the new openApi structure

* fix : fix the shemar deps in example
takumi-earth pushed a commit to earthlings-dev/rmcp that referenced this pull request Jan 27, 2026
…tprotocol#258)

* chore(deps): update schemars requirement from 0.8 to 0.9

* fix: adapt the new openApi structure

* fix : fix the shemar deps in example
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-config Configuration file changes T-core Core library changes T-dependencies Dependencies related changes T-examples Example code changes T-handler Handler implementation changes T-test Testing related changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants