-
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/revamp #23530
Docs/revamp #23530
Conversation
0e42d65
to
95c2cf6
Compare
Deploy preview for dagster-docs ready! Preview available at https://dagster-docs-7ob7nywzg-elementl.vercel.app Direct link to changed pages: |
637707c
to
805b36a
Compare
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
❌ Deploy Preview for dagsterapidocs failed.
|
Deploy preview for dagster-docs-next ready! ✅ Preview Built with commit e1e2eff. |
Deploy preview for dagster-docs-beta ready! ✅ Preview Built with commit 8de4fa2. |
lgtm |
## Summary & Motivation This adds the new beta docs site into our main repo. This is still a work in progress and does not affect the deployment of our existing docs. This work was originally undertaken 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. --------- Co-authored-by: colton <[email protected]>
## Summary & Motivation This PR beefs up our Vale implementation with some custom rules and vocabularies. The README for this will be updated in a subsequent PR. ## How I Tested These Changes eyes, local
## Summary & Motivation This PR addresses comments in #23754 ## How I Tested These Changes
137ed67
to
a1e46b0
Compare
d8f990d
to
46b8bc1
Compare
## Summary & Motivation Misspelling in readme. ## How I Tested These Changes N/A
Brings in #23754 and #23758 into the master branch for docs-beta styling. --------- Co-authored-by: Erin Cochran <[email protected]>
We shouldn't have `.yarn/install-state.gz` checked in.
Summary & Motivation
How I Tested These Changes