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

fix encoding problems on windows with not using default encoding but … #893

Closed
wants to merge 2 commits into from

Conversation

brizjin
Copy link

@brizjin brizjin commented Jul 15, 2019

Solves encoding problem on windows with using fetch, pull, push commands. It similar to issues: 332, 237, explanation of problem I found here in accepted answer. Also was encoding problem with calling repo.branches, quite simillar, file open in default locale encoding (cp-1251) for me, and getting branches names was broken.

@Byron
Copy link
Member

Byron commented Jul 20, 2019

Thanks a lot for your efforts!
Unfortunately there is some breakage, maybe you can have a look.
Thank you

@Byron
Copy link
Member

Byron commented Jan 6, 2021

As part of spring cleaning, I am closing this PR as it was opened more than half a year ago while still being open due to unresolved or remarks.

If this is a mistake or any other action should rather be taken, please let me know in the comments or create a new PR.
Thanks for your understanding and the work that undoubtedly went into this.

@Byron Byron closed this Jan 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants