Skip to content

ci: prevent sponsor block workflow from running on forks#10641

Merged
jasonsaayman merged 1 commit intoaxios:v1.xfrom
shaanmajid:fix/sponsor-block-fork-guard
Apr 3, 2026
Merged

ci: prevent sponsor block workflow from running on forks#10641
jasonsaayman merged 1 commit intoaxios:v1.xfrom
shaanmajid:fix/sponsor-block-fork-guard

Conversation

@shaanmajid
Copy link
Copy Markdown
Contributor

@shaanmajid shaanmajid commented Apr 3, 2026


Summary by cubic

Prevent the Sponsor Block GitHub Actions workflow from running on forks by gating the job to the main axios/axios repo. This avoids noisy failures and saves CI minutes.

Description

  • Add job-level guard: if: github.repository == 'axios/axios' in update-sponsor-block.yml.
  • Reason: forks don’t have required secrets/permissions, causing failed or unnecessary runs.
  • Scope: CI-only; no runtime code changes.

Testing

  • No unit tests added (workflow-only change).
  • Expected behavior: on forks, the job is marked “skipped”; on axios/axios, it runs normally.
  • Manual check: open a fork PR and confirm the workflow shows “skipped”.

Written for commit e60dd09. Summary will update on new commits.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Copy link
Copy Markdown
Member

@jasonsaayman jasonsaayman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks 🔥

@jasonsaayman jasonsaayman added the commit::ci The PR is related to CI label Apr 3, 2026
@jasonsaayman jasonsaayman merged commit e4bd759 into axios:v1.x Apr 3, 2026
28 of 30 checks passed
@shaanmajid shaanmajid deleted the fix/sponsor-block-fork-guard branch April 3, 2026 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commit::ci The PR is related to CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants