Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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

Closed
brizjin wants to merge 2 commits into
gitpython-developers:mastergitpython-developers/GitPython:masterfrom
brizjin:masterbrizjin/GitPython:masterCopy head branch name to clipboard
Closed

fix encoding problems on windows with not using default encoding but …#893
brizjin wants to merge 2 commits into
gitpython-developers:mastergitpython-developers/GitPython:masterfrom
brizjin:masterbrizjin/GitPython:masterCopy head branch name to clipboard

Conversation

@brizjin

@brizjin brizjin commented Jul 15, 2019

Copy link
Copy Markdown

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

Byron commented Jul 20, 2019

Copy link
Copy Markdown
Member

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

@Byron

Byron commented Jan 6, 2021

Copy link
Copy Markdown
Member

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.