-
-
Notifications
You must be signed in to change notification settings - Fork 31.4k
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
bpo-34166: Use tobytes instead of tostring and use with statement for file handling #8367
Conversation
Thanks @tirkarthi for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.6, 3.7. |
Sorry @tirkarthi and @serhiy-storchaka, I had trouble checking out the |
(cherry picked from commit a692efe) Co-authored-by: Xtreak <tirkarthi@users.noreply.github.com>
GH-8368 is a backport of this pull request to the 3.6 branch. |
(cherry picked from commit a692efe) Co-authored-by: Xtreak <tirkarthi@users.noreply.github.com>
GH-8369 is a backport of this pull request to the 3.7 branch. |
Strangely enough there weren't any conflicts for 3.7 branch when I used cherry_picker command but still @miss-islington failed to make a PR.
|
Changes made :
tobytes
instead oftostring
.Tested using a sample .po file as in https://bugs.python.org/msg322074
I think this needs a backport to 3.7 and 3.6. I haven't added a NEWS entry since the changes seem trivial let me know if it's needed.
Thanks
https://bugs.python.org/issue34166