-
Notifications
You must be signed in to change notification settings - Fork 361
Convert Python objects to booleans based on their Python truth value #435
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
Conversation
|
Don't forget to update https://github.com/oracle/python-cx_Oracle/blob/master/doc/src/release_notes.rst !! |
Done. |
anthony-tuininga
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. If you can make those few changes I'll merge it.
Signed-off-by: Alex Henrie <[email protected]>
Creating a DB using Oracle Cloud's free https://www.oracle.com/cloud/free/ tier can be handy for testing. |
Thanks for the suggestion! I created an Oracle Cloud account today and was able to confirm that the new tests work as they should :-) |
|
Thanks, @alexhenrie! |
|
I made a few tweaks after examining it a bit more. Hopefully the changes are clear but if you have questions, yell out! |
Looks good to me. Thanks! |
I added tests for this, but since I don't have a local Oracle database for testing, I can't be 100% sure that they work.