-
-
Notifications
You must be signed in to change notification settings - Fork 577
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
[i18nIgnore] Fix links #1039
[i18nIgnore] Fix links #1039
Conversation
|
✅ Deploy Preview for astro-starlight ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
size-limit report 📦
|
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.
👏
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.
Took a bit of time to test them all but clearly way less than it took you to fix them all. Impressive work 🤯
Thanks for the reviews and testing! Yeah, this is definitely something it would be nicer to stay on top of with each PR in a CI check. @HiDeoo Saw you added the extra context HiDeoo/starlight-links-validator#9 — thank you! |
Yeah, I also played a little bit with it this morning as I had my head in the code while releasing a new version. The logic should be relatively straightforward altho I think the smart move is to wait for plugins so I can turn the integration into a plugin and easily have access to the Starlight config to check if the site is multilingual or not and check which locales are configured. |
Absolutely. I’d also love a rule to enforce use of same-language links, which were a bunch of what I’d caught… wait I guess I should make a feature request 😁 |
What kind of changes does this PR include?
Description
/es/
pages, all relative links should be to/es/
versions of pages).\]\(/(?!ru)
indocs/src/content/docs/ru/
).P.S. I considered if we could integrate the link checker into the docs site but HiDeoo/starlight-links-validator#9 is blocking us currently from doing that I think.