-
Notifications
You must be signed in to change notification settings - Fork 145
Python 3 Compatibility - Rebased Fork #57
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
Conversation
…ll tests now pass.
…passes tests using both python2 and python3.
|
|
|
@geeknam did you get the chance to review this? |
|
👍 I was thinking about doing similar PR. :)) |
|
Thanks guys, can we get the coverage & health up? Who wants to volunteer to do this? :) |
|
I believe the priority should be making a release rather than waiting for people to improve health indicator & tests. All current tests pass so the library should be good to go for a release with this branch. This is my 0.02 |
|
Thanks for your input @rakanalh. Landscape caught an undefined variable |
|
|
|
In my opinion, using Benefits:
|
|
I agree... but many people along with myself need this to be Py3 compliant as soon as possible. As for requests, i also agree. I prefer using requests over urllib. Could you leave this as an open issue for someone to pick up and create a PR for that? |
|
#60 fixes this |
This PR is a rebase of @avendael's work on Python3 compatibility to the latest version of develop.
Thank you @geeknam, @avendael