Results 893 comments of Sebastian Thiel

Thanks so much for your tips! I will be switching to the latest commit in #995 moving forward. I also took a closer look at the cache_capacity configurable, and think...

Thanks for your reply and your sympathy regarding the computer issue - now I am working on a 5 year old MBPro with a proper keyboard, and enjoying it tremendously...

Yes, I wanted to create the docs myself, but didn't expect them to be personalized due to the public nature of the project. Maybe one could apply just a few...

Thanks a lot for posting this here! I had a quick look and believe the reason is the following: When [asking for `urls`](https://github.com/gitpython-developers/GitPython/blob/24cd6dafc0008f155271f9462ae6ba6f0c0127a4/git/remote.py#L528-L534) git will be invoked to ask for...

I think it does make sense that their output is consistent, with `urls` being correct in this case. Since calling implementing `url` like `next(urls)` is not the same both in...

Sorry for the `url` link, indeed it should rather have been a link to [this location](https://github.com/gitpython-developers/GitPython/blob/24cd6dafc0008f155271f9462ae6ba6f0c0127a4/git/remote.py#L430:L430), which is the fallback in case the attribute does not exist. In that case,...

Thanks for the research! This means that if we would use `"rt"` instead, the issue would somewhat magically disappear. The reason the file is read as binary is that otherwise,...

Awesome! `rt` is implemented by Python, not by the OS, it seems. And unicode-escaping needs backslashes to insert codepoints like `\uffff`, among others. All that seems to be done so...

Sometimes one has to dig deeper :D, so I found [this portion of the git source code](https://github.com/git/git/blob/master/config.c#L594:L594) which seems to mean that it's simply decoding these with `\\ ➡ \`...

[In theory](https://github.com/gitpython-developers/GitPython/blob/6df6d41835cd331995ad012ede3f72ef2834a6c5/git/repo/base.py#L961), it should pass through any `kwarg` to clone as well. Just recently there was [the `multi_options`](https://github.com/gitpython-developers/GitPython/blob/6df6d41835cd331995ad012ede3f72ef2834a6c5/git/repo/base.py#L995) flag added, which might end up doing what you want. Even though...