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.

Sign up for GitHub

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

Jump to bottom

adds lazy loading of cs50.SQL #5

Merged
merged 5 commits into from
Feb 25, 2017
Merged

adds lazy loading of cs50.SQL #5

merged 5 commits into from
Feb 25, 2017
+48 −13

Conversation

dmalan
Copy link
Member

@dmalan dmalan commented Jan 24, 2017

  • So that most of the library can be used when SQLAlchemy is not installed, adds lazy loading of cs50.SQL.
  • Adds comments to sql.py.

dmalan added 4 commits January 24, 2017 00:26
@dmalan
added SQLAlchemy for Python2 too

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
GPG key ID: 4AEE18F83AFDEB23
Expired
Verified
Learn about vigilant mode
d1c6c93
@dmalan
tweaked tests to test imports of cs50.SQL

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
GPG key ID: 4AEE18F83AFDEB23
Expired
Verified
Learn about vigilant mode
d7ec5e6
@dmalan
added lazy loading of cs50.SQL, added comments
5472755
@dmalan
updated version
00f4b66
@dmalan dmalan merged commit 46aeb6b into master Feb 25, 2017
@dmalan dmalan deleted the import-hooking branch February 25, 2017 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@glennholloway glennholloway

@kzidane kzidane

@brianyu28 brianyu28

Assignees

@glennholloway glennholloway

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@dmalan @glennholloway