-
-
Notifications
You must be signed in to change notification settings - Fork 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
Fix dependencies in requirements-dev.txt #5070
Conversation
Turns out that the requirements-dev.txt is generated from conda env files in the pre-commit. |
Codecov Report
@@ Coverage Diff @@
## main #5070 +/- ##
===========================================
+ Coverage 61.11% 78.43% +17.31%
===========================================
Files 130 130
Lines 24461 24461
===========================================
+ Hits 14950 19185 +4235
+ Misses 9511 5276 -4235
|
(we should probably end up moving that to an issue and not block this PR, but I wanted to start the I want to take this opportunity and rethink all these conda environments we have. I understand they are helpful and convenient to create our dev environments and they are also used for ci. However, at least in my case, the extra work and burden on updating 4 (8 now) nearly equivalent files with a pre-commit check that seems to only partially work (i.e. pandas has a lower bound in 3.7 and 3.8 but not in 3.9 nor in requirements-dev.txt) is clearly larger than the benefit I get. Moreover, issues with these files, installation and pre-commit blocked #5060. #5062 was only needed because of #5060 being blocked. I think there are several issues at hand regarding all this.
Ideas for possible solutions:
|
+1 on revisiting how we bundle requirements. |
e5d26b5
to
b820b7d
Compare
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 have removed unnecessary dependencies from the ci only files to simplify conda's job and hopefully speed up a bit the env creation process
I have also removed recommonmark and nbsphinx from everywhere they appeared. We don't use any of the two anymore, nor we have done since #4759. Please don't add them again, this has to be the 3rd or 4th PR where I have removed them from the requirements.
The requirements-dev were updated in PR - #4932. After the merge, RTD could not build docs because of missing dependencies. So, this PR reverts the changes in requirements-dev.txt.
I am not sure of the differences between
environment-dev-py37.yml
andenvironment-test-py37.yml
. Butsphinx_design
andsphinx-notfound-page
are not present in environment-test* yml files. So, is there a way to generate these yml files from requirements-dev.txt?