-
Notifications
You must be signed in to change notification settings - Fork 1.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
[docs-beta] Merge the new beta Docs site #23748
Conversation
This adds some base linting styles using Vale. These are a collection of both suggestions and errors that will raise when run against our new docs site.
This is a single commit for the work done on the new docs revamp project. In order to keep the commit history clean, changes from the docs/revamp branch were squashed into a single commit.
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.
can remove along with other docusaurus images
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.
good call
Deploy preview for dagster-docs-beta ready! ✅ Preview Built with commit 98fb6f3. |
Summary & Motivation
This adds the new beta docs site into our main repo. This is still a work-in-progress, and does not affect our existing docs deployment. This work was originally undertook in the #23530 branch, however, I have brought it in as a squashed self-contained PR to keep the history clean.
How I Tested These Changes
Built locally, BK, and Vercel.