Skip to content

DOC: add to README so example works under Python 3.6 #21

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

Merged
merged 1 commit into from
May 8, 2019

Conversation

stsievert
Copy link
Contributor

What does this PR implement?
It adds decode_responses=True in the example in README.rst as mentioned in #8 (comment) (which took me too long to find after copy/pasting the example with Python 3.6.5 and rejson-py 0.3.0).

So no TypeError under Python 3.6
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling cd0f92c on stsievert:patch-1 into ba4adf5 on RedisLabs:master.

@gkorland gkorland requested a review from itamarhaber March 5, 2019 08:04
@rjcortese
Copy link

I am another victim of this minor detail.... please review and merge maintainers.

Copy link
Collaborator

@itamarhaber itamarhaber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine.

itamarhaber added a commit to itamarhaber/rejson-py that referenced this pull request May 8, 2019
In lieu of RedisJSON#21 as merging appears disabled.
@gkorland gkorland merged commit a428662 into RedisJSON:master May 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants