Skip to content
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

Updated the code to work on newer versions of python (3.5+) fixes #1 #2

Merged
merged 2 commits into from
Jul 11, 2016

Conversation

jNullj
Copy link
Contributor

@jNullj jNullj commented Jul 11, 2016

changes i made:

print is now required to be used as print('test') insted of print 'test'
and ConfigParser was changed to configparser

fixed some spacing/tab issues

and decleared bytes with b to avoid join problems (join expects string unless there is b before the calling string and should recive all parameters as bytes)

i tested those changes with python 3.5.2 (last updated Jun 25, 2016. )

print is now required to be used as print('test')
and ConfigParser was changed to configparser

fixed some spacing/tab issues

and decleared bytes with b to avoid join problems (join expects string unless there is b before the calling string and should recive all parameters as bytes)
@bentasker
Copy link
Owner

Thanks! Seems I missed the notification of the original issue being raised.

Based on quick test seems to work OK for me, so merging

@bentasker bentasker merged commit 1ee3d0d into bentasker:master Jul 11, 2016
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.

2 participants