-
Notifications
You must be signed in to change notification settings - Fork 286
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
updated installing instructions to use pip #5273
Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #5273 +/- ##
=======================================
Coverage 89.32% 89.32%
=======================================
Files 89 89
Lines 22390 22390
Branches 5374 5374
=======================================
Hits 20000 20000
Misses 1640 1640
Partials 750 750 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
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.
Thanks @tkknight! Is there more to do on this (still in Draft)?
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.
@tkknight Lovely change 👍
Thanks for showing the docs and its infrastructure so much attention 💯
* upstream/main: updated installing instructions to use pip (SciTools#5273) remove sphinx-panel config as now longer needed (SciTools#5272)
* upstream/main: Two negating pins for dependencies - libnetcdf and numpy (SciTools#5274) [pre-commit.ci] pre-commit autoupdate (SciTools#5275) updated installing instructions to use pip (SciTools#5273) remove sphinx-panel config as now longer needed (SciTools#5272)
🚀 Pull Request
Description
Updated the installing instructions to use:
pip install --no-deps --editable .
instead of:
python setup.py develop
Using pip is considered better practice, also the old way shows a couple of strong warnings:
Consult Iris pull request check list