-
Notifications
You must be signed in to change notification settings - Fork 8
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
Only trim trailing index from links if it is the full segment #80
Only trim trailing index from links if it is the full segment #80
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
🦋 Changeset detectedLatest commit: fb19163 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
5d03224
to
354d57e
Compare
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 catch, thanks for the thorough pull request 🙌
Everything looks good to me 🚀
Previously the replacement
replace(/index$/, '')
was called when normalizing file paths which would result in links like/packages/spec/module_index
to be listed as invalid because it would get normalized to/packages/spec/module_
.This PR fixes that by only replacing
index
at the end of a file path if it is the full segment.