-
-
Notifications
You must be signed in to change notification settings - Fork 430
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
Add support redis-py 3.0.0 #344
Conversation
Extending |
@michael-k Good call! In the latest version I've updated tox & Travis to test against both redis-py 2.x and 3.x. Thanks for the suggestion. |
django_redis/client/default.py:37:41: W504 line break after binary operator tests/test_backend.py:548:21: W504 line break after binary operator tests/test_backend.py:550:22: W504 line break after binary operator tests/test_backend.py:604:22: W504 line break after binary operator
Session data is always text.
- Make CacheKey inherit from str to fix exception "Only bytes, strings and numbers (ints, longs and floats) are acceptable for keys and values." - .exists() now returns and integer, so update .has_key(). - Update test matrix to include tests for both redis-py 2.x and the latest version. Fixes #342
@TheOldMop I've incorporated your test fixes from #335 so we can verify all tests pass. Thanks! |
@jdufresne I didn't notice this on the queue, but not sure if you want to look at terencehonles@a4f7781 for the |
Thanks. Looks mostly identical. |
Nice work! Thank you all. |
str
to fix exception "Only bytes, strings and numbers (ints, longs and floats) are acceptable for keys and values.".exists()
now returns and integer, so update.has_key()
.Fixes #342