sandr01d

Yes, that's what I expected. The git version that comes with jammy does have this option, so it works there. Given that git 1.25 (and Ubuntu 20.04) is quite old...

Ok, you both have convincing arguments. We should make it work for git 2.25.1.

> Do you think, l I should add it to the documentation the same way FORGIT_CHECKOUT_BRANCH_BRANCH_GIT_OPTS is explained ? Yes, that sounds useful :+1:

Thanks for the insights @carlfriedrich. While the solution you provided is certainly an interesting way to solve this, I think replacing preview functions with injected code goes beyond configuration since...

> isn't it already possible ? > > FORGIT_BLAME_PAGER='bat --color=always -l log' git forgit blame @ccoVeille this would color everything using the same syntax (log), but ideally we would have...

I agree with you @carlfriedrich. The AUR package already has `delta` as an optional dependency and we also list it on our README. The only thing to look out for...

> Also, you could mention delta as a recommended dependency We already mention it as an optional dependency on the README [here](https://github.com/wfxr/forgit?tab=readme-ov-file#-optional-dependencies) and it's set as an optional dependency in...

I think this would be a big improvement. > I also think this may be easier once https://github.com/wfxr/forgit/pull/326 lands, as it breaks the code into a bunch of small simple...

> What are your thoughts on the advantages this would bring? The advantage would be to keep the main repo free of clutter. I expect that there would be quite...

That sounds like a reasonable start @wfxr. In case we notice regressions in other areas we could still expand our coverage.