-
Notifications
You must be signed in to change notification settings - Fork 18
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
conform to Records' API #55
Comments
I think both projects should add compatibility with the other, the shortcut methods of postgres.py and the chaining of Records are both useful. |
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@kennethreitz has launched Records, "SQL for Humans™". Given the relative star count (~1,500 in 6 days vs. 36 in 3 years), it seems that the Python world now has a de facto DB-API 3.0—and it's not ours. :o)
Records is database-agnostic (cf. kennethreitz/records#41). So far it seems to me that there's value in continuing with Postgres.py because of the ORM layer. The proposal here is to change our top-level
run
/one
/all
API to conform to what @kennethreitz has come up with:So we'd essentially rename
run
toquery
, and chainall
(andone
, which Records doesn't have—yet?) onto the result.The text was updated successfully, but these errors were encountered: