Skip to content

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
Member

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 Oct 23, 2023
@tiangolo tiangolo changed the title 📝 Clarify description of in-memory SQLite database Oct 23, 2023
@tiangolo
Copy link
Member

Great, thank you! 🚀 🍰

@tiangolo tiangolo enabled auto-merge (squash) October 23, 2023 20:44
@tiangolo tiangolo merged commit 80fd7e0 into fastapi:main Oct 23, 2023
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

3 participants