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

Port the lib to Python 3 #3

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

Port the lib to Python 3 #3

wants to merge 2 commits into from

Conversation

mpiraux
Copy link

@mpiraux mpiraux commented Aug 3, 2017

Hello,

I ported the lib to Python 3 quite easily by simply running the 2to3 tool. I also fixed a bug preventing the test suite to complete without errors.

Maybe this should not be merged directly into the master branch, but judging by the recent activity, or rather the lack thereof, I doubt that the Python2 branch will receive any future updates. Actually, moving the Python2 version in a separate branch would be the best choice in my opinion.
I intend on developing the lib further in the future. In fact I already started rewriting its internals while keeping the API untouched, but that will be the matter of another pull request.

@mpiraux
Copy link
Author

mpiraux commented Aug 20, 2017

Hello @pietercolpaert,

Any updates on this PR ? I think an up to date Python API for IRail is quite interesting, and even more since it got access to the real-time feeds :)
While the absolute gain from using this API rather than implementing it inside a Python project is quite low right now, as I said, I intend on developing it further in the future to make it more pythonic and maybe to provide more advanced functionalities.

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.

1 participant