Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Character encoding fix courtesy github user nathanapples #3108

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

YipYup
Copy link
Contributor

@YipYup YipYup commented Apr 11, 2018

Retrying pull request taking @nathanapples fixes, and fixing the styling issue reported by Travis regarding the pep8 error.

@YipYup YipYup changed the title Yip yup patch 1 Character encoding fix courtsey github user nathanapples Apr 11, 2018
@YipYup
Copy link
Contributor Author

YipYup commented Apr 11, 2018

Replaces #3009

Courtesy @nathanapples

@YipYup YipYup changed the title Character encoding fix courtsey github user nathanapples Character encoding fix courtesy github user nathanapples Apr 11, 2018
@YipYup
Copy link
Contributor Author

YipYup commented Apr 30, 2020

@AdeHub bump

@YipYup
Copy link
Contributor Author

YipYup commented May 26, 2024

@rembo10

@rembo10
Copy link
Owner

rembo10 commented May 26, 2024

Hey, sorry, I think this is a bad idea.
https://stackoverflow.com/questions/3828723/why-should-we-not-use-sys-setdefaultencodingutf-8-in-a-py-script

I think better if we address the core issue if there is a problem in the encoding/decoding

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants