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

Upgrade code for Python 3.7 #15

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

Conversation

Johnson145
Copy link

Not sure whether this repository gets still maintained, but I've updated the code to work with Python 3.7.

@SahilAshar
Copy link

it doesn't look like this repo is actively maintained anymore - but thanks for conversion! It might be useful to make an issue as well that links to this PR.

@JeanFred
Copy link
Member

Hi! Sorry − I completely missed this post last August…

Wow, I did not even think that anyone would care much about that repo! Thanks for working on it, i’m more than happy to merge this.

@JeanFred
Copy link
Member

Ok, I took some time to fix the build on master and the tests are passing again. However there are test failures under Python3 − would need investigation

@SahilAshar
Copy link

There was also an issue with one of the imports in thumbnaildownload.py -> it needs to be import urllib.request instead of just import urllib. That got it working for me.

@JeanFred JeanFred marked this pull request as draft March 19, 2021 15:28
@JeanFred JeanFred marked this pull request as ready for review March 19, 2021 15:28
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.

3 participants