CI: Skip jobs on forks by oscargus · Pull Request #30118 · matplotlib/matplotlib · GitHub
Skip to content

CI: Skip jobs on forks#30118

Merged
QuLogic merged 1 commit into
matplotlib:mainfrom
oscargus:forkci
Jun 4, 2025
Merged

CI: Skip jobs on forks#30118
QuLogic merged 1 commit into
matplotlib:mainfrom
oscargus:forkci

Conversation

@oscargus

Copy link
Copy Markdown
Member

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

@oscargus oscargus added CI: Run cibuildwheel Run wheel building tests on a PR CI: testing CI configuration and testing labels May 29, 2025
@tacaswell

Copy link
Copy Markdown
Member

@tacaswell tacaswell modified the milestones: v3.11.0, v3.10.4, v3.10-doc May 29, 2025
@tacaswell

Copy link
Copy Markdown
Member

and we might as well backport this to all of our active branches.

Comment thread .github/workflows/cibuildwheel.yml Outdated
Comment thread .github/workflows/cibuildwheel.yml Outdated
@oscargus

Copy link
Copy Markdown
Member Author

you must open a PR or remove the check

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...).

@oscargus

Copy link
Copy Markdown
Member Author

@QuLogic QuLogic closed this Jun 4, 2025
@QuLogic QuLogic reopened this Jun 4, 2025
@QuLogic QuLogic merged commit 5535df4 into matplotlib:main Jun 4, 2025
66 of 73 checks passed
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Jun 4, 2025
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Jun 4, 2025
QuLogic added a commit that referenced this pull request Jun 4, 2025
…118-on-v3.10.3-doc

Backport PR #30118 on branch v3.10.3-doc (CI: Skip jobs on forks)
QuLogic added a commit that referenced this pull request Jun 5, 2025
…118-on-v3.10.x

Backport PR #30118 on branch v3.10.x (CI: Skip jobs on forks)
@oscargus oscargus deleted the forkci branch June 5, 2025 08:10
@ksunden ksunden mentioned this pull request Aug 7, 2025
5 tasks
@ksunden ksunden mentioned this pull request Aug 30, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI: Run cibuildwheel Run wheel building tests on a PR CI: testing CI configuration and testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants