-
-
Notifications
You must be signed in to change notification settings - Fork 689
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
β Fix typos in docs/databases.md
and docs/tutorial/index.md
#35
Conversation
π Docs preview for commit 24fa99b at: https://61b8d5f35a3dba130fdddaac--sqlmodel.netlify.app |
Codecov Report
@@ Coverage Diff @@
## main #35 +/- ##
=======================================
Coverage 97.58% 97.58%
=======================================
Files 182 182
Lines 6054 6054
=======================================
Hits 5908 5908
Misses 146 146 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
π Docs preview for commit 064241d at: https://630a84c46be883468d605bd9--sqlmodel.netlify.app |
π Docs preview for commit d520f4f at: https://630a86976be88346db605cd6--sqlmodel.netlify.app |
π Docs preview for commit 101f2ab at: https://630a88d895131e42f7ab9898--sqlmodel.netlify.app |
docs/databases.md
and docs/tutorial/index.md
π Docs preview for commit 4867538 at: https://630a8e4aa5277f4830062922--sqlmodel.netlify.app |
Thanks for the interest! Some of the changes were mainly stylistic and depending on personal taste, so I reverted those, but kept and kept the two typo fixes. Thanks for the effort! π° |
Trying not to step on any toes in terms of existing docs/typo fixes in other PRs, so I've done my best and gone through all the PRs related to the docs. π
Summary
docs/tutorial/index.md
anddocs/databases.md
Β #5 and β Fix multiple typos and some rewordingΒ #22 (I didn't duplicate those fixes in this PR)docs/tutorial/index.md
anddocs/databases.md
Β #5 didn't seem to be addressed in existing PRs, so that fix has also been made intutorial/fastapi/simple-hero-api.md