Skip to content

Conversation

HarHarLinks
Copy link
Contributor

@HarHarLinks HarHarLinks commented Jul 29, 2022

@HarHarLinks HarHarLinks requested a review from a team as a code owner July 29, 2022 20:47
Half-Shot added a commit to Half-Shot/backend-meta that referenced this pull request Aug 1, 2022
As found in matrix-org/matrix-hookshot#430, the linkifying in GitHub included the `.` and so the anchor didn't work.

Also, it turns out people sometimes miss the format we expect and do their own thing. Rather than trying to make the regex even more lenient, I'd rather just suggest the format we expect explicitly.
DMRobertson pushed a commit to matrix-org/backend-meta that referenced this pull request Aug 1, 2022
As found in matrix-org/matrix-hookshot#430, the linkifying in GitHub included the `.` and so the anchor didn't work.

Also, it turns out people sometimes miss the format we expect and do their own thing. Rather than trying to make the regex even more lenient, I'd rather just suggest the format we expect explicitly.
HarHarLinks and others added 3 commits August 27, 2022 11:53
Co-authored-by: Will Hunt <will@half-shot.uk>
Co-authored-by: Will Hunt <will@half-shot.uk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants