-
Notifications
You must be signed in to change notification settings - Fork 163
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
Run tests with Python 3.9 #182
Conversation
Codecov Report
@@ Coverage Diff @@
## master #182 +/- ##
==========================================
- Coverage 94.69% 94.28% -0.41%
==========================================
Files 25 32 +7
Lines 1507 1942 +435
Branches 203 258 +55
==========================================
+ Hits 1427 1831 +404
- Misses 48 76 +28
- Partials 32 35 +3
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We go for it and if something breaks later on we fix it.
Status
READY
Background context
Some libraries are dropping support to Python 3.6, such as pip, so we need to prepare to support later versions of Python.
Description of the changes proposed in the pull request
Add test in CircleCI to run in an image with Python 3.9. A new version doesn't have to be released, because only test files were modified.
Where should the reviewer start?
File
.circleci/config.yml
then the Python files.Remaining problems or questions
The step "Install package" takes about 10 minutes to run in Python 3.9, probably due to
scikit-learn
(in Python 3.6 they take ~1 minute); and the unit tests take ~15 minutes (in Python 3.6 they take ~1 minute).