Skip to content
This repository was archived by the owner on Jan 20, 2020. It is now read-only.

Added Python 2 and Python 3 compatibility. #29

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

penkin
Copy link

@penkin penkin commented Mar 15, 2017

Had to add mock for the Python 2 test as well. All tests pass in both Python2 and Python3 environments.

Christopher Penkin added 7 commits March 15, 2017 23:08
Added six and __future__ imports to better compatibility between
python2 and python3 environments.
Remove the excessive need of six functions that are not really needed.
Fixed up the auth tests.
Fixed the base64.v64encode to return a string and not bytes objects.
Added mock lib for Python2 and reverted the os.envion.get to remove the
default kwarg that is not in Python2.
Made the code a little cleaner and easier to read.
…test as well. All tests pass in both Python2 and Python3 environments.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant