-
Notifications
You must be signed in to change notification settings - Fork 3
Issue 4 merge latest upstream changes #5
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.
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
base: main
Are you sure you want to change the base?
Issue 4 merge latest upstream changes #5
Conversation
Co-authored-by: Marcelo Trylesinski <[email protected]>
…ol#474) Co-authored-by: Claude <[email protected]>
Co-authored-by: David Soria Parra <[email protected]> Co-authored-by: Basil Hosmer <[email protected]> Co-authored-by: ihrpr <[email protected]>
…FastMCP servers (modelcontextprotocol#540) Co-authored-by: ihrpr <[email protected]>
…odelcontextprotocol#637) Co-authored-by: yangben <[email protected]>
I just wanted to note that nearly all tests pass with these changes. Here's the tail-end of the output from running
However, when I run the test suite for the official upstream, all tests pass:
And here's what I get when I run the tests on the
The two failing tests on the EMQX fork pass in the official upstream. Most of the failures for the EMQX fork (with and without the PR applied) display an error message saying: |
…extprotocol#1186) Co-authored-by: Felix Weinberger <[email protected]>
…l#1285) Co-authored-by: Olivier Schiavo <[email protected]>
3a4c1c3
to
29b9dae
Compare
This PR aims to bring the EMQX fork up-to-date with the official upstream (ref issue #4).