-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
upgrade to docusaurus 3 #9820
upgrade to docusaurus 3 #9820
Conversation
|
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@docusaurus/core@2.4.3, npm/@mdx-js/react@1.6.22, npm/docusaurus-preset-openapi@0.6.4 |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
in docusaurus 3 this is now a hard error, not just a warning
d6accc5
to
f5df52a
Compare
fixes performing the 'execute' action when pressing enter
I'll keep this updated as we merge new stuff, but this one would be really useful to get a review on sooner rather than later as it touches a lot of things. |
🚀 Updated review app: https://pr-9820-badges-shields.fly.dev |
🚀 Updated review app: https://pr-9820-badges-shields.fly.dev |
I've decided I'm just going to go rogue and merge this one. |
Closes #9816
This PR upgrades
and I've made all the necessary formatting changes to all of the documentation to make MDXv3 happy, display everything sensibly and fix all the
validateDOMNesting
warnings (mostly<p> cannot appear as a descendant of <p>
and<a> cannot appear as a descendant of <a>
warnings).This PR is changing a lot at once, but I have split things out into commits so the functional changes are easy to separate from the minor alterations to documentation markup.
Relevant background reading:
One of the things that docusaurus 3 should allow us to do is migrate the frontend from CJS to ESM, but I've decided not to do that here because we don't have to do it in order to upgrade and the PR is already big. I'm going to punt that change to a follow up issue.