New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Require `http(s)://` prefix to render links #6395

Merged

siggisim merged 1 commit into master from siggi-dev-branch-20240417-093101

Apr 17, 2024

Member

siggisim commented Apr 17, 2024

We can relax this later, but the current regex is a bit too lenient and is rendering links all over the place.


          Require http(s):// prefix to render links

62d2fca

siggisim requested review from bduffany and jdhollen

April 17, 2024 16:31

jdhollen approved these changes

View reviewed changes

siggisim merged commit 37d188e into master

10 of 18 checks passed

siggisim deleted the siggi-dev-branch-20240417-093101 branch

April 17, 2024 16:35

jdhollen pushed a commit that referenced this pull request


          Require http(s):// prefix to render links (#6395)

e24483f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment