__str__ must return type str

Cogniac / cogniac-sdk-py

Python SDK for Cogniac Public API

Apache License 2.0

5 stars 6 forks source link

Closed sergii-bond closed 3 years ago

sergii-bond commented 3 years ago

resolve #89

Didn't find any other way than by checking python version.

sergii-bond commented 3 years ago

please check Sean's comment https://github.com/Cogniac/cogniac-sdk-py/issues/89#issuecomment-812089425, using 'six' library is another way. We also have dict iteritems() in app.py which in python3 is replaced by items(), 'six' library claims to address that as well.

wskish commented 3 years ago

six does seem to popular for dealing with these cases; lets go with the six approach

sergii-bond commented 3 years ago

I'll let Sean make a pull request with six, he is working on that.