-
Notifications
You must be signed in to change notification settings - Fork 346
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 support and unit tests #539
base: master
Are you sure you want to change the base?
Conversation
@laxdog Testing is always appreciated. Is this still on your radar? If not, we should consider this PR stale and close it out... |
@laxdog if I fix conflicts, is this at least stable enough to merge? I am looking to starting a major overhaul which needs Python 3 around the end of this month. |
I'm going to redo it. There are a few things I wasn't aware of or weren't available when I started it last time. Don't delete the branch just yet, I still want to compare code from it. |
I'm assuming here we're dropping python2 support if you're wanting something from 3 @Evanito ? I'm happy to do that and it makes it much cleaner if so, just checking before I rewrite a lot of this. |
I'm excited to see some renewed interest in this. 900 stars is quite the achievement. Keep it going! |
@laxdog Yes, async in particular. 👍 Since this will be breaking backwards compatibility, merge this into the dev branch instead of master. So users only have to manually switch once at a later time. |
What branch do you want this PR raised against @Evanito ? |
@laxdog python36 |
Is this branch working? Python2 will lose support soon and it's very urgent to update to Python 3. |
Add support for python 3 (Only tested on 3.6 so far)
Description
Python 2 will eventually be discontinued. Also this allows for async and other functionality eventually if we ever move off 2.7
Unit tests were needed anyway, so I thought this was a good chance to add them to each module.
I've ported all the code (I think) to python 3, whilst still supporting python 2. There are still a lot of tests to write however to check it's working.
So far I've about half of Configuration done, MarketAnalysis was already done.
Opening this now for visibility in case anyone has comments or wants to help out.
You can run the test with
pytest -v
TESTING STAGE
Currently running this branch and testing.
Types of changes
Checklist: