Skip to content

fix: add status field in Resource class and Resource as a return type in CallToolResult#758

Closed
bzsurbhi wants to merge 2 commits intomodelcontextprotocol:mainfrom
bzsurbhi:feature/async-support
Closed

fix: add status field in Resource class and Resource as a return type in CallToolResult#758
bzsurbhi wants to merge 2 commits intomodelcontextprotocol:mainfrom
bzsurbhi:feature/async-support

Conversation

@bzsurbhi
Copy link
Copy Markdown

@bzsurbhi bzsurbhi commented May 19, 2025

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

@felixweinberger felixweinberger self-assigned this Jul 14, 2025
Copy link
Copy Markdown
Contributor

@felixweinberger felixweinberger left a comment

Choose a reason for hiding this comment

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

Hi @bzsurbhi thanks for your contribution!

I notice this PR has been in draft for a while and doesn't have a description to explain what the intent of the changes is.

If this is still something you'd want to work on would you be able to add the description according to the template?

Requesting changes for now to put back in your queue.

@felixweinberger felixweinberger added needs more work Not ready to be merged yet, needs additional follow-up from the author(s). needs motivation labels Sep 5, 2025
@felixweinberger felixweinberger removed their assignment Oct 14, 2025
@maxisbey
Copy link
Copy Markdown
Contributor

Closing due to inactivity. If you'd like to continue this work, please open an issue first to discuss the approach.

AI Disclaimer

@maxisbey maxisbey closed this Dec 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs more work Not ready to be merged yet, needs additional follow-up from the author(s).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants