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

Python 3 compatability. #3

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

Conversation

ResidentMario
Copy link

Makes this library Python 3 compatible.

@ResidentMario
Copy link
Author

I've created a test repository to demonstrate that the change to api.py is library cross-version compatible.

I've also tested out running python setup.py in both 2 and 3, since those changes are more iffy, and it works in both versions.

@afeierman
Copy link

This looks great (thanks ResMario for porting), but why remove so much from the README?

@ResidentMario
Copy link
Author

Not sure how that happened, but I think the developer API instructions for sign up have changed (gotten simpler) so that section is now errant? Not sure.

@aepyornis
Copy link
Contributor

aepyornis commented May 9, 2018

hey @ResidentMario -- thanks for providing a python3 version of this.

I tried installing your fork and it fails to load -- any ideas?

import_error_geoclient png

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