-
-
Notifications
You must be signed in to change notification settings - Fork 30.9k
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
gh-122191: Fix test_warnings failure if run with -Werror #122222
gh-122191: Fix test_warnings failure if run with -Werror #122222
Conversation
__spec__.loader is now required in the module globals (see pythongh-86298).
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.
Seems like my suggestion had some typos. Slightly concerning that it passed anyway.
Co-authored-by: Alex Waygood <[email protected]>
Thanks @serhiy-storchaka for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13. |
…nGH-122222) __spec__.loader is now required in the module globals (see pythongh-86298). (cherry picked from commit 9b4fe9b) Co-authored-by: Serhiy Storchaka <[email protected]>
Sorry, @serhiy-storchaka, I could not cleanly backport this to
|
Thank you for your review @AlexWaygood. |
GH-122256 is a backport of this pull request to the 3.13 branch. |
…pythonGH-122222) __spec__.loader is now required in the module globals (see pythongh-86298). (cherry picked from commit 9b4fe9b) Co-authored-by: Serhiy Storchaka <[email protected]>
GH-122257 is a backport of this pull request to the 3.12 branch. |
…22222) (GH-122256) __spec__.loader is now required in the module globals (see gh-86298). (cherry picked from commit 9b4fe9b) Co-authored-by: Serhiy Storchaka <[email protected]>
__spec__.loader
is now required in the module globals (see gh-86298).test_warnings
fails if run with-Werror
#122191