`check-vcs-permalinks` - Add support for different branch names

Not all repositories use master for their branch name so it would be great if this hook could support other branch names.

Preferably, this would match any blob link that doesn’t contain a full hash (so only matching 0-f with len of 40 or 64) but if you don’t consider that to be a good solution, then being able to specify a list of branches to match would also work, though it would be a lot more likely to lead to non-permalinks not getting matched.

[asottile]

YAGNI imo

[Jackenmen]

afaik this is irrelevant for git which uses SHA1 – [A-Fa-f0-9]{7,40} seems fine to me

Right now yes, but Git team is making a continuous effort towards SHA256 support (Git 2.29 introduced experimental support for it, I’m not following it that closely to know how far they are with it now) so I figured it might be worth supporting it already since I don’t think it would cause any issues.

“contains a .” is what pre-commit uses as a decent heuristic for a tag – it’s of course impossible to get this correct

That seems fine, do you think it’s worth providing a custom regex option, or is that just an unnecessary complication (until proven it’s actually needed anyway)?

---

I can look into PRing this if that’s fine with you.