Broken link in README: ./docs/migrations/migrating-to-v5.md (migration guides moved under docs/reference/) #3566
Replies: 2 comments
|
Verified your fix end to end, and it is correct on all three counts. The target resolves — ### Using custom equality functions such as `shallow`so I also checked whether the move left other casualties, since a directory rename usually does. It did not, in either direction: Every other doc link in the README resolves — And searching the whole repo for the stale So it really is a one-line change with no follow-ups hiding behind it, which is worth saying in the PR — a reviewer looking at a link fix after a directory move will reasonably wonder whether it is the only one, and it is. |
Uh oh!
There was an error while loading. Please reload this page.
Summary
In
README.md(line 120, currentmain@ b126c33), thecreateWithEqualityFnnote links to:That path does not exist — the migration guides live under
./docs/reference/migrations/, so the link 404s when reading the README on GitHub.Suggested fix
The anchor is unchanged and still matches the
### Using custom equality functions such as \shallow`heading indocs/reference/migrations/migrating-to-v5.md`.Scope check
I resolved every relative
docs/link in the README against the repo at b126c33: this is the only one that is still broken, and all of them resolve after this one-line change. Prettier (prettier --list-different README.md) is clean with the change applied.This is the same class of link rot as #3537 / #3542 and #3518 / #3519.
Note
I have the fix ready on a branch and would have opened a PR, but PR creation is currently limited to repository collaborators, so I am reporting it here instead. Happy to open the PR if the restriction is lifted, or feel free to take the diff as-is.
All reactions