-
Notifications
You must be signed in to change notification settings - Fork 338
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.
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
✅ Add link checker #2189
✅ Add link checker #2189
Conversation
b7a4b92
to
3762419
Compare
3762419
to
bb4340b
Compare
d513a91
to
ccfd02d
Compare
8f1238c
to
bac9c1e
Compare
15e9b28
to
c272fe1
Compare
a932ab6
to
8f104f9
Compare
896990e
to
85dbc2c
Compare
e078472
to
8476cb5
Compare
Including non-standard syntax that hurt parsers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! That's quite a massive PR. I can't wait to see the automatic link checker live!
A few proposed changes here and there, but nothing big either.
- Final built pages are working as expected.
- The information is correct.
- The information is comprehensive.
- The information is concise.
- The information is clear.
- All links work, including to section headers.
✅ Okey to merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes, looks better now 🎉
Why
We want to ensure all our links are still valid.
Closes #2049
What's changed