-
Notifications
You must be signed in to change notification settings - Fork 13
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
Review and add tips from notion doc #1053
Conversation
Deploying docs with
|
Latest commit: |
a8d7185
|
Status: | ✅ Deploy successful! |
Preview URL: | https://0b1b9bb4.docodile.pages.dev |
Branch Preview URL: | https://docs-838.docodile.pages.dev |
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.
Some copyedit suggestions to use more active text avoid stating opinions about what is easy or difficult. If a suggestion doesn't help, please ignore it. I'll go ahead and approve so that you won't be blocked either way.
Co-authored-by: Matt Linville <[email protected]>
Thanks I think I addressed most everything |
Pulls relevant info from best practices docs linked in https://wandb.atlassian.net/browse/DOCS-838, and adds it to various areas in the docs. Left over from the Jan docs team 🐛 🔨
Confirm that tests pass and this change is ready for review:
hugo serve
vale ./content