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
Fixed #35090 -- Deprecated registering URL converters with the same name. #17703
Conversation
THank you for the work, though the ticket is not accepted yet. I think we should wait to see some input from others to accept it. One change that your PR will require is using the deprecation policy. We would need to warn at first, and then two versions later we can raise an exception. Read the policy and check out some recent PRs that have deprecated other features. |
c4a2688
to
e82c59b
Compare
e82c59b
to
5357484
Compare
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.
@salvo-polizzi Thanks 👍 I left comments.
e7870c5
to
6b715af
Compare
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.
Thanks for the continued work. I’ve left some comments on the tests.
6b715af
to
82958df
Compare
Thanks @adamchainz for review |
82958df
to
5600f8c
Compare
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.
Minor formatting adjustment but otherwise LGTM!
5600f8c
to
6e1ece7
Compare
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.
@salvo-polizzi Thanks 👍 I pushed final edits 👍
@adamchainz @shangxiao Thanks for reviews.
Ticket 35090