Upgrade to latest starlight#647
Conversation
Use new features for a small simplification of the nav JSON. Also address some minor type errors in the JSON.
Deploying localstack-docs with
|
| Latest commit: |
972fd82
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://ac5efb67.localstack-docs.pages.dev |
| Branch Preview URL: | https://upgrade-starlight-0-39.localstack-docs.pages.dev |
|
In the future, let's keep PRs scoped to the task. I would prefer we open a separate PR for adding the mailhog extension doc to the nav, that is not relevant to upgrading starlight. :) |
Understood. It came up in this one because I was trying to simplify some of the nav JSON using Starlight. That was one of the sections that could be simplified (and thus how I noticed the nav item was missing). |
|
definitely get it, sometimes i see a small change i want to fix and it seems almost silly to open up a whole other pr for it we can go ahead and merge this one if you like, i just want us to try to for the most part do cleaner prs :D |
The nav structure changed in Starlight 0.39 and this fixes issues in the current structure
2 participants
Use new features for a small simplification of the nav JSON. Also address some minor type errors in the JSON.
Fixes DOC-236