-
-
Notifications
You must be signed in to change notification settings - Fork 573
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
Fixing issues related to tox-conda #6521
Conversation
…the Python version" This reverts commit e75e4da.
b73fc04
to
2a924a2
Compare
2a924a2
to
9d7836f
Compare
9d7836f
to
0388a0a
Compare
@@ -35,6 +35,7 @@ dependencies: | |||
# Testing | |||
- hypothesis | |||
- pytest | |||
- pytest-arraydiff |
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 also needs to be enabled by passing --arraydiff
to the pytest command.
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'm adding it to the environment file to be explicit. It's already a dependency in truth - the pytest mark would not be defined otherwise - but we never encountered an issue because the dependency will always be satisfied due to pytest-astropy
having it as a dependency. Thus, adding it to the environment file doesn't actually change the environment because it's already installed anyway.
Actually enabling it our tox commands requires me to fix that one test, which I will get to in a separate PR.
For our cron CI builds between the 25th and the 27th this past week,
pytest
went from 7.1.3 to 7.2.0 andtox
went from 3.26.0 to 3.27.0, but alsotox-conda
went from 0.9.2 to 0.10.0. Probably due to the newtox-conda
release, but maybe due to something else, ourpy39-conda
build is once again running Python 3.10 instead of the intended 3.9. This PR reverts my workaround in #6494, and as predicted in that PR, the newtox-conda
release no longer needs to be worked around.Also, adding
pytest-arraydiff
as a direct dependency (it was already an indirect dependency throughpytest-astropy
)Also, pinning
pytest
andpytest-xdist
a la #6508 (see also #6505)Also, adding
hvpy
as a dependency, viapip
until it is available onconda-forge
.hvpy
has a dependency (pydantic
) that is not otherwise installed in the conda environment, so temporarily explicitly installpydantic
.