-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 CI for Python3.9 #4647
Run CI for Python3.9 #4647
Conversation
Also, `dvc[hdfs]` install will be a no-op till we have a pyarrow wheel for 3.9. Note that this does not add an entry to the classifier claiming it supports 3.9, that will be added when 3.9 gets released (or, wait for pyarrow to be released).
Also, fix the python version regex for `3.9.0b5` like versions.
libxml does not have wheels for Python39. We do compile them on Linux and Mac
These test failures look completely unrelated. Looks like CI is not that stable. |
from tests.unit.output.test_local import TestLocalOutput | ||
|
||
|
||
@pytest.mark.skipif(PY39, reason=PYARROW_NOT_AVAILABLE) |
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.
I guess we could've created a custom marker, but this will do too.
@@ -1,4 +1,10 @@ | |||
import os | |||
import sys | |||
|
|||
# FIXME: Search and replace these from the tests if pyarrow wheel is available |
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.
This will get lost. How about a ticket? (Yeah, that one could be lost too 🙂 ).
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.
It won't, as I will come to this next week, as soon as Python3.9 gets released.
Windows tests for Python3.8 was failing intermittently. Note: Also removed the realpath wrap around self.root_dir as it's already a realpath when `Git` is instantiated.
if not path_isin(os.path.realpath(gitignore), self.root_dir): | ||
raise FileNotInRepoError(path) |
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.
This exception here does not seem very precise,
Thank you! 🙏 |
Run CI for Python3.9, skips hdfs tests for 3.9
d673e2a
Also,
dvc[hdfs]
install will be a no-op till we havea pyarrow wheel for 3.9.
Note that this does not add an entry to the classifier
claiming it supports 3.9, that will be added when 3.9
gets released (or, wait for pyarrow to be released).
❗ I have followed the Contributing to DVC checklist.
📖 If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here.
Thank you for the contribution - we'll try to review it as soon as possible. 🙏