Tricky to get rid of the lookbehind here, I probably have to refactor the entire function to use a different approach to prefix URLs with a protocol.
Ok, I found a way to do it without a regex.
As a bonus, the code is also easier to understand now (and that's coming from me π)
monkey
!m kellnerd
BrainzBot
You're doing good work, kellnerd!
kellnerd
Thanks, I'm just filling the PR template :)
Is it intentional that we not only prefix the href with https:// but also the displayed link? I think it would look better to show it as simply www.example.com nevertheless.
s/look better/be more intuitive/
That wasn't easily doable with the old approach, but it is with the new one
monkey
I agree
kellnerd
Ok, I will push that change as a second commit π