-
Notifications
You must be signed in to change notification settings - Fork 229
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
reqs: Move pyrevolve to optionals and introduce testing-only reqs #2096
Conversation
e7a10ca
to
14153aa
Compare
47792f6
to
24a3b62
Compare
Codecov Report
@@ Coverage Diff @@
## master #2096 +/- ##
==========================================
- Coverage 87.48% 87.15% -0.34%
==========================================
Files 220 220
Lines 39280 39300 +20
Branches 5113 5113
==========================================
- Hits 34365 34251 -114
- Misses 4346 4478 +132
- Partials 569 571 +2
|
2aa42d1
to
c682d38
Compare
a5611fc
to
56a16d1
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.
LGTM now. Thanks a lot.
pip install -e . | ||
pip install matplotlib | ||
pip install -e .[tests] | ||
pip install matplotlib pyrevolve |
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.
Does that ensure that we install the latest versions?
e.g. this is why we "cannot" see the maptlolib issue in our CI
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.
pip
always install the latest compatible version.
No description provided.