-
-
Notifications
You must be signed in to change notification settings - Fork 143
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
federation-api: Deprecate v1/send_join
and v1/send_leave
#1556
Conversation
0405318
to
3d173c6
Compare
Did we deprecate the entire module for other deprecated endpoints? I think it makes sense, I just can't remember seeing it before :) |
I don't think there are other deprecated endpoints atm. With a string search I only get the tests. The old deprecated client-api sync endpoints are not implemented. |
07947fb
to
4aa30ea
Compare
It looks like CI doesn't like having tests in a deprecated module… Unless I'm misunderstanding the issue. |
We may be hitting rust-lang/rust#47238 |
I've got it to work by moving |
Maybe we should move it to be an integration test, to keep the tree clean? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
According to a spec clarification
Otherwise cargo check always complains that we are using a deprecated const. See rust-lang/rust#47238
2b66ffb
to
95d410a
Compare
According to a spec clarification.
This required to allow to declare an endpoint as both stable and deprecated in Matrix 1.0.
Preview: https://pr-1556--ruma-docs.surge.sh