Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
CI: Skip jobs on forks#30118
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
CI: Skip jobs on forks#30118
Conversation
Only downside is that if you want to use CI to check wheel buliding youmust open a PR or remove the check, but I think that is a reasonable trade off to avoid burning the commute on every push (there is a reason we only do it on PRs in an opt-in basis). |
and we might as well backport this to all of our active branches. |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Although that wasn't part of the plan, I think that one can push the branch to the matplotlib repo to get that. Since the people with push access are the same that are most likely to do that, I think it should be manageable (although a bit of behavior change as most seems to use their private forks. Personally, I always open a PR anyway as I had not thought about the alternative...). |
Btw, it seems like GitHub has disabled workflows on forks by default(?). Some recently cloned repos has a text that says that they are disabled. Makes sense.
Seems to have happened quite recently though. (Or maybe there is some setting for the main repo? But it seems to correlate with when I forked them, at least a quick check says that.) |
PR summary
Tests are skipped when pushing changes to a fork, but some other jobs are run. Especially Build CI wheels is a bit unnecessary to run on forks. Also, the CodeQL job is executed periodically on forks, which may lead to emails, but if nothing else waste of compute resources.
PR checklist