-
-
Notifications
You must be signed in to change notification settings - Fork 31.9k
gh-100746: Improve test_named_expressions.py
#116713
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
Conversation
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.
Looks great, thanks for getting more test coverage on this!
Backport to 3.12 will be as-is, while 3.11 will be much smaller, because 3.11 does not have new binding logic from 3.12 |
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!
Thanks @sobolevn for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12. |
(cherry picked from commit 25684e7) Co-authored-by: Nikita Sobolev <[email protected]>
GH-116747 is a backport of this pull request to the 3.12 branch. |
(cherry picked from commit 25684e7) Co-authored-by: Nikita Sobolev <[email protected]>
GH-116748 is a backport of this pull request to the 3.11 branch. |
Two major changes:
pass
where possible and instead used an assert for what we expect to happendict
has key and value, we need to be sure that both work correctly.test_named_expressions.py
#100746