fix: MultipleChoiceField use ordered sort#9735
fix: MultipleChoiceField use ordered sort#9735fbozhang wants to merge 20 commits intoencode:mainfrom
Conversation
auvipy
left a comment
There was a problem hiding this comment.
please also add unit tests to verify this and make the CI green
I’ve added the tests and the CI is green now. |
browniebroke
left a comment
There was a problem hiding this comment.
Seems sensible to me 👍🏻
However, it will have to wait for 3.17 as I see this as potentially disruptive for some downstream users. Left a small suggestion
Good point — waiting for 3.17 sounds like the right call. Appreciate the suggestion! |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This modification has the following advantages: