Jelmer Vernooij
Jelmer Vernooij
Hi @ggtylerr any news on this?
LGTM. I'll wait a few days to give @ix5 a chance to object. I'll merge otherwise.
We should do another release at some point; first we should fix the failing tests though.
What version are you trying to build?
FWIW Dulwich now (0.24+) has support for line ending handling and git attributes.
I've fixed the signature warning and have rebased to resolve merge conflicts. @mcepl the other warnings you're listing happen in "main" for me as well, they're not specific to this...
@zsol any news on this?
Please rebase - the ruff issues are fixed in master.
> These commits are already based on [4de310b](https://github.com/jelmer/dulwich/commit/4de310b6676856434391ab57b55e0e4489967abe) which seems to be the latest at present? Whoops, the PR actually got stuck because of a spurious test failure. Current HEAD...
Changes LGTM, just needs a "ruff format" run?