Skip to content

Conversation

@talolard
Copy link

I added python3.x support.
Changed exception syntax and used the six.moves library for urllib,urllib2 and urlparse.
I added six as a dependency for the installation.
Tests passed.

@talolard talolard closed this Jul 11, 2015
@talolard talolard reopened this Jul 11, 2015
@talolard
Copy link
Author

Tests don't pass.
Some of the ids are wrong I think.
I wasn't sure if this is a change in ids/results or if I broke something.
I pulled a fresh copy of the master which also failed tests, so I'm assuming I didn't break anything.

@ocelma
Copy link
Owner

ocelma commented Jul 13, 2015

Yes, I think some IDs in the tests are wrong. I had to change it before.
Feel free to fix that, as well :-)
Then, I'm happy to merge and pull your request.

BTW, will the library still be working for 2.X? (or, at least for 2.7?)

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