When building scheduled wheels, run count-dists as a successful no-op#9244
When building scheduled wheels, run count-dists as a successful no-op#9244radarhere wants to merge 1 commit intopython-pillow:mainfrom
Conversation
|
Why does 'sdist' does not run when scheduled? To only upload wheels to scientific-python-nightly-wheels and avoid a ~50 MB zip? Or perhaps unhook the 'count-dists' check from the scientific-python-nightly-wheels flow? 
|
It turns out I suggested it a year ago. I expect I just thought it wasn't necessary to upload it. If you'd like to though, I have no objection. I've created #9248
All other things being equal, I think it would be better to know if we suddenly start uploading extra wheels to scientific-python-nightly-wheels. |
I checked a few others at https://anaconda.org/scientific-python-nightly-wheels/ and didn't see any sdist, so probably better not to upload a fairly large file. And the clue's in the (scientific-python-nightly-wheels) name.
Agreed. How about we use #9248 and do something like |
Sure. I've updated the PR, just changing the 'pattern' used for matching in scientific-python-nightly-wheels-publish. |
|
Closed in favour of #9248. |
2 participants
Alternative to #9248
Normally, our scheduled Wheels jobs update wheels to scientific-python-nightly-wheels - https://github.com/python-pillow/Pillow/actions/runs/18331606908
Our most recent scheduled Wheels jobs did not - https://github.com/python-pillow/Pillow/actions/runs/18437781793
This is because #9239 made 'scientific-python-nightly-wheels-publish' require 'count-dists', which requires 'sdist', but 'sdist' does not run when scheduled.
This PR suggests always running 'sdist', but if the jobs are scheduled, skip all the steps within it.