Add codeberg link parsing

[p00f]

No description provided.

[p00f]

@dandavison can you take a look? Codeberg links are the same format as GitHub so there isn't much change

[dandavison]

Hi @p00f, thanks and sorry for being unresponsive! Can you send me shell commands to clone a codeberg repo so that I can try this out?

I think that what I'd ideally like is to add the ability for users to configure the remote link parsing regex themselves. (Is there any chance you feel like working on that??) That was what made me hesitate here. It is true that I have only barely heard of codeberg (or sourcehut), but of course I don't want to privilege the well-known git hosting companies over smaller ones. Nevertheless I would ideally like a more scalable solution that doesn't involve delta hard-coding this logic, even if, in practice, I am not about to be inundated with thousands of pull requests adding regular expressions parsing URLs from the world's lesser-known git hosting companies :)

[p00f]

sure, git clone https://codeberg.org/dnkl/fuzzel

That was what made me hesitate here

I was about to write "don't worry, I don't think there are other FOSS-focused hosts" 😄

Is there any chance you feel like working on that??

Yes! (Can the "configuration" for the major ones i.e. sourcehut and codeberg be built-in along with github and gitlab?)

How would I write tests for this?

[dandavison]

OK, merged this, thanks very much!

(Can the "configuration" for the major ones i.e. sourcehut and codeberg be built-in along with github and gitlab?)

Sure. But, actually, I wonder whether maybe this is more work than it's worth. Can you think of a good configuration interface/API for users to specify this parsing logic? We have --file-transformation where users supply a sed-style transformation, but it would need to be more complicated than that. I'm thinking perhaps there are more important things to work on in delta (e.g. line wrapping in the non-side-by-side cases #1176 #299)

[p00f]

In #1176 and #299 the issue is that blame and non-side-by-side are not wrapped, correct?

[dandavison]

That's right. @th1000s implemented word-wrapping for the side-by-side case but we don't yet use any of that wrapping machinery in non-side-by-side contexts.