Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃摑 Clarify description of in-memory SQLite database in docs/tutorial/create-db-and-table.md #601

Merged
merged 3 commits into from Oct 23, 2023

Conversation

SimonCW
Copy link
Contributor

@SimonCW SimonCW commented May 31, 2023

Tried to make the docs more clear by avoiding confusion about the in-memory url for sqlite.

Why:
I was quite confused because I didn't notice the difference between the sqlite urls with a filename and without a filename, using only two "//". Also, I first thought the in-memory database would be provided by SQLAlchemy, but only the url format is determined by SQLAlchemy.

@SimonCW
Copy link
Contributor Author

SimonCW commented May 31, 2023

Tried to fix the CI here #602, but ultimately failed because poetry-version-plugin isn't working anymore.

@antont
Copy link

antont commented Jun 2, 2023

Tried to fix the CI here #602, but ultimately failed because poetry-version-plugin isn't working anymore.

This seems to have poetry updates.

#603

@SimonCW
Copy link
Contributor Author

SimonCW commented Jun 3, 2023

Yeah, I didn't see this. I since then got #602 to work and unblock CI .

I assume tiangolo is currently quite busy with the pedantic V2 migration for FastAPI

@tiangolo tiangolo added the docs Improvements or additions to documentation label Oct 22, 2023
@tiangolo
Copy link
Owner

tiangolo commented Oct 23, 2023

I confused two PRs, sorry! Reopening this.

@tiangolo tiangolo closed this Oct 23, 2023
@tiangolo tiangolo reopened this Oct 23, 2023
@tiangolo tiangolo changed the title Clarify description of in-memory sqlite 馃摑 Clarify description of in-memory SQLite database Oct 23, 2023
@tiangolo tiangolo changed the title 馃摑 Clarify description of in-memory SQLite database 馃摑 Clarify description of in-memory SQLite database in docs/tutorial/create-db-and-table.md Oct 23, 2023
@tiangolo
Copy link
Owner

Great, thank you! 馃殌 馃嵃

@tiangolo tiangolo enabled auto-merge (squash) October 23, 2023 20:44
@tiangolo tiangolo merged commit 80fd7e0 into tiangolo:main Oct 23, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants