Add compatibility properties to new PG Range class #8927
Conversation
There was a problem hiding this comment.
OK, this is sqla-tester setting up my work on behalf of zzzeek to try to get revision 8b9e7b7 of this pull request into gerrit so we can run tests and reviews and stuff
|
New Gerrit review created for change 8b9e7b7: https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/4266 |
|
mike bayer (zzzeek) wrote: I reworded some of the docstrings and also implemented a quick pep-484 typing on top (lots of type: ignores though). View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/4266 |
|
mike bayer (zzzeek) wrote: the rebase didnt work? not sure what that was about View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/4266 |
|
mike bayer (zzzeek) wrote: recheck weird things going on View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/4266 |
|
mike bayer (zzzeek) wrote: OHHH it's my fault, there's crap in the directory View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/4266 |
|
mike bayer (zzzeek) wrote: recheck View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/4266 |
|
Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/4266 has been merged. Congratulations! :) |
This adds a bunch of properties to new PG Range class for compatibility with other implementations, providing a more similar API to access emptiness and bounds status.