-
-
Notifications
You must be signed in to change notification settings - Fork 330
Require setuptools
64.0.0+
#1193
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
Require setuptools
64.0.0+
#1193
Conversation
CC: @joshmoore @jakirkham |
setuptools
setuptools
64.0.0+
Thanks Saransh! 🙏 Updated the PR title to make this a bit clearer (hope that is ok 🙂) |
Codecov Report
@@ Coverage Diff @@
## main #1193 +/- ##
=======================================
Coverage 99.99% 99.99%
=======================================
Files 35 35
Lines 14136 14135 -1
=======================================
- Hits 14135 14134 -1
Misses 1 1
|
Finally catching up with my email backlog, I'd like to add a request for help with keeping the release.rst entries up-to-date for PRs. It makes releases SO much easier. |
I will update It might also be nice to add an action that labels every PR with something like "needs changelog" if |
Thanks!
@Saransh-cpp, definitely! Several of the ongoing tasks around |
I completely forgot that
setuptools
introduced editable installs inv64.0.0
. This means that the installation will fail if a developer, who hassetuptools
<v64.0.0
installed locally, tries to installZarr
withpip install -e .
right now.With this PR, the local version of
setuptools
will be automatically updated to the compatible version.For more information see this tweet - https://twitter.com/juanluisback/status/1557734536586625025?t=rzVtT-91AwzSOvBHanCJ9g&s=19
TODO: