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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support SQLAlchemy 2.0 #174

Merged
merged 7 commits into from
Oct 24, 2023
Merged

Support SQLAlchemy 2.0 #174

merged 7 commits into from
Oct 24, 2023

Conversation

rongxin-liu
Copy link
Contributor

No description provided.

@rongxin-liu
Copy link
Contributor Author

Tested with finance and movies problem sets.

@Jelleas
Copy link

Jelleas commented Oct 24, 2023

Hi Rongxin! I just ran into this issue too, would love a merge on this PR :)

Quick double check, shouldn't this one be wrapped with sqlalchemy.text() too?

cursor.execute("PRAGMA foreign_keys=ON")

@rongxin-liu
Copy link
Contributor Author

Hi Rongxin! I just ran into this issue too, would love a merge on this PR :)

Quick double check, shouldn't this one be wrapped with sqlalchemy.text() too?

cursor.execute("PRAGMA foreign_keys=ON")

Hey, here we're executing a SQL statement directly using a database cursor (sqlite3 backend), not the SQLAlchemy engine so I think this usage does not need to be or should be replaced with sqlalchemy.text().

@rongxin-liu rongxin-liu merged commit fcc8cb9 into main Oct 24, 2023
@rongxin-liu rongxin-liu deleted the SQLAlchemy-2.0 branch October 24, 2023 17:45
@Jelleas
Copy link

Jelleas commented Oct 24, 2023

🥳 thanks!

LEDMann added a commit to LEDMann/CESBooks that referenced this pull request Feb 1, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
… this issue cs50/python-cs50#174
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants