add an example for homebrew style file upload#1637
add an example for homebrew style file upload#1637SamYuan1990 wants to merge 10 commits intomodelcontextprotocol:mainfrom
Conversation
There was a problem hiding this comment.
please don't commit PDFs to this repo, they're large binary files and often considered not secure to open unless trusted
There was a problem hiding this comment.
ok, I will remove the pdf file this week. it's mid night my local time today.
Signed-off-by: SamYuan1990 <yy19902439@126.com>
Signed-off-by: SamYuan1990 <yy19902439@126.com>
Signed-off-by: SamYuan1990 <yy19902439@126.com>
Signed-off-by: SamYuan1990 <yy19902439@126.com>
16e5fef to
3a25a01
Compare
|
hi @maxisbey , as you have my word, I just updated this example as remove pdf files and pass CI checks. Would you like to give it a review? |
|
I won't be reviewing this as it's not something we can merge. As mentioned in the linked issue, this would require a SEP to be accepted and merged into a draft spec of MCP. Unfortunately I don't have time to do code reviews for things that aren't able to be merged :( The SEP I'm guessing this is following, modelcontextprotocol/modelcontextprotocol#1306, hasn't yet been accepted and would need to be before any maintainers would look at adding this to the Python SDK. Going to switch this PR to a draft for now. |
Motivation and Context
an example for modelcontextprotocol/modelcontextprotocol#1306 from modelcontextprotocol/modelcontextprotocol#1522
address #1590
How Has This Been Tested?
see document as readme of this PR.
Breaking Changes
n/A
Types of changes
Checklist
Additional context