-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Multiple broken anchors #5630
Comments
We'd accept a PR to fix this |
I'll make a start on this. For ease of reviewing I will submit multiple PRs. There are also false positives in the list, which will need looking at to see if they can be avoided. |
All broken anchor links have now been fixed. Thanks to @jennifer-shehane for her patience in merging all the related PRs! 🙂 The errors in anchor links had built up over the years and there was no automatic checking in place until Docusaurus added this functionality recently. The final
|
|
@MikeMcC399 Thank you! That was a lot of fixes. |
You're welcome! 🙂 |
Description
When I executed
npm run build
I expected no broken anchors to be reported.URL of Issue(s)
https://docs.cypress.io/
Steps to replicate
git clone https://github.com/cypress-io/cypress-documentation cd cypress-documentation npm install npm run build
Additional Information
^2.4.1
to3.1.1
.Bad anchor links generally cause reader confusion, as they will typically call up the intended page and then display it at the wrong place, leaving the reader to search and / or scroll for the relevant information.
npm run build log (original state)
The text was updated successfully, but these errors were encountered: