Do not run scheduled wheel jobs on forks by Yay295 · Pull Request #8254 · python-pillow/Pillow
Navigation Menu
{{ message }}
python-pillow / Pillow Public
- Notifications You must be signed in to change notification settings
- Fork 2.4k
Merged
Do not run scheduled wheel jobs on forks#8254
hugovk merged 1 commit intopython-pillow:mainfrom
Do not run scheduled wheel jobs on forks#8254
hugovk merged 1 commit intopython-pillow:mainfrom
Conversation
Copy link Copy Markdown
Contributor
Skip two more wheel builds on schedule, and only run the publish jobs in the main repository.
Fixes #8253.
Yay295
mentioned this pull request
Copy link Copy Markdown
Member
radarhere
commented
Jul 21, 2024
radarhere commented
Jul 21, 2024Skip two more wheel builds on schedule
Could you explain why you've made this change?
Copy link Copy Markdown
Contributor Author
Yay295
commented
Jul 21, 2024
Yay295 commented
Jul 21, 2024I assumed you missed them in lysnikolaou#1.
Copy link Copy Markdown
Member
radarhere
commented
Jul 21, 2024
radarhere commented
Jul 21, 2024No, the aarch64 wheels were skipped because they weren't building free threaded wheels, and sdist was skipped because that doesn't build wheels at all.
Yay295
force-pushed
the
patch-2
branch
from
495888e to
56a0420
Compare
Copy link Copy Markdown
Contributor Author
I think I understand it now. This new change should work correctly.
Basically: Don't build anything on schedule unless it's running in this repository.
radarhere approved these changes Jul 22, 2024
radarhere
changed the title
Build wheels less often
Do not run scheduled wheel jobs on forks
hugovk
merged commit
22ef8df
into
python-pillow:main
Copy link Copy Markdown
Member
hugovk
commented
Jul 22, 2024
hugovk commented
Jul 22, 2024Thanks!
Yay295
deleted the
patch-2
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants