Skip to content

example: add sampling example and test #289

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
4t145 merged 1 commit into from
Jun 30, 2025
Merged

example: add sampling example and test #289

4t145 merged 1 commit into from
Jun 30, 2025

Conversation

@jokemanfire
Copy link
Collaborator

@jokemanfire jokemanfire commented Jun 27, 2025 •
edited
Loading

Motivation and Context

1.add simpling example and test
2.ignore .vscode in .ignore
3.change some name in example

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 27, 2025 14:11
@github-actions github-actions bot added T-dependencies Dependencies related changes T-test Testing related changes T-config Configuration file changes T-examples Example code changes labels Jun 27, 2025
@jokemanfire
Copy link
Collaborator Author

jokemanfire commented Jun 27, 2025

#224

@jokemanfire jokemanfire mentioned this pull request Jun 27, 2025
Open
@4t145 4t145 changed the title example: add simpling example and test example: add sampling example and test Jun 30, 2025
4t145
4t145 requested changes Jun 30, 2025
View reviewed changes
@jokemanfire
example: add simpling example and test
c5cd67a 
Signed-off-by: jokemanfire <hu.dingyang@zte.com.cn>
@github-actions github-actions bot added the T-documentation Documentation improvements label Jun 30, 2025
@jokemanfire jokemanfire requested a review from 4t145 June 30, 2025 09:49
@4t145 4t145 merged commit 1b3db89 into modelcontextprotocol:main Jun 30, 2025
10 of 11 checks passed
@github-actions github-actions bot mentioned this pull request Jul 2, 2025
Merged
bug-ops pushed a commit to bug-ops/rust-sdk that referenced this pull request Jul 3, 2025
@jokemanfire @bug-ops
example: add simpling example and test (modelcontextprotocol#289)
416a8b0 
Signed-off-by: jokemanfire <hu.dingyang@zte.com.cn>
bug-ops pushed a commit to bug-ops/rust-sdk that referenced this pull request Jul 9, 2025
@jokemanfire @bug-ops
example: add simpling example and test (modelcontextprotocol#289)
f98ccea 
Signed-off-by: jokemanfire <hu.dingyang@zte.com.cn>
bug-ops pushed a commit to bug-ops/rust-sdk that referenced this pull request Jul 20, 2025
@jokemanfire @bug-ops
example: add simpling example and test (modelcontextprotocol#289)
7756887 
Signed-off-by: jokemanfire <hu.dingyang@zte.com.cn>
bug-ops pushed a commit to bug-ops/rust-sdk that referenced this pull request Jul 23, 2025
@jokemanfire @bug-ops
example: add simpling example and test (modelcontextprotocol#289)
e0062a7 
Signed-off-by: jokemanfire <hu.dingyang@zte.com.cn>
takumi-earth pushed a commit to earthlings-dev/rmcp that referenced this pull request Jan 27, 2026
@jokemanfire
example: add simpling example and test (modelcontextprotocol#289)
b1f5d12 
Signed-off-by: jokemanfire <hu.dingyang@zte.com.cn>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@4t145 4t145 4t145 approved these changes

Assignees

No one assigned

Labels

T-config Configuration file changes T-dependencies Dependencies related changes T-documentation Documentation improvements T-examples Example code changes T-test Testing related changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jokemanfire @4t145