Skip to content
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

Bump minor version + correct encoding errors people face with cp1252 #332

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Moooog
Copy link

@Moooog Moooog commented Oct 20, 2022

Bump minor version to 0.4.12 + correct encoding errors people face with cp1252 (cp850 works but not recommended)

UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d in position 1477: character maps to <undefined>
Ignore cp1252 encoding error (cp850 works as default but isnt recommended so ignoring the error is preferred and will successfully write requirement file)
Add error="ignore" to both open()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant