-
Notifications
You must be signed in to change notification settings - Fork 443
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
chore: add best practice page #1790
Conversation
3167681
to
8d75503
Compare
Co-authored-by: Ansgar Mertens <[email protected]>
b980c6d
to
7823e6b
Compare
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.
We're very close on this - I'm excited! See what you think of these suggestions. Please push back in any cases where my edits misrepresent the meaning :)
Co-authored-by: Laura Pacilio <[email protected]>
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.
Just fixing a few very minor typos. After you merge these in, please feel free to merge this PR! Great work on this!! I think this is an excellent start at this best practices page that we can iterate on over time!
Co-authored-by: Laura Pacilio <[email protected]>
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Document best practices, of course there are more, but we can evolve this page over time :)
Closes #974, #399