Skip to content

ci: skip deploy-preview job for fork PRs#13584

Merged
ilhan007 merged 2 commits into
mainfrom
ci/skip-deploy-preview-for-forks
May 26, 2026
Merged

ci: skip deploy-preview job for fork PRs#13584
ilhan007 merged 2 commits into
mainfrom
ci/skip-deploy-preview-for-forks

Conversation

@ilhan007
Copy link
Copy Markdown
Collaborator

@ilhan007 ilhan007 commented May 26, 2026

Summary

  • Fork PRs can't access the Netlify and bot token secrets, so the deploy-preview job always fails on them and shows up as a red check on the PR.
    For eample: docs: fix typo on using assets page #13581
  • Gate the job on github.event.pull_request.head.repo.full_name == github.repository so it runs only for same-repo PRs and is skipped for forks.
  • Skipped checks render as ✓ neutral and don't block merge, so contributors from forks no longer see a misleading failure.

Test plan

  • Open a PR from a fork and confirm the Deploy Preview / deploy-preview check appears as skipped, not failed.
  • Open a PR from a branch in this repo and confirm the deploy preview still runs and posts the Netlify URL.

ilhan007 and others added 2 commits May 26, 2026 12:40
Fork PRs don't have access to the Netlify and bot token secrets, so the
deploy step always failed and surfaced as a failing check on the PR.
Gate the job on the head repo matching the base repo so it's skipped
(not failed) for forks. Skipped checks don't block merge.
@ilhan007 ilhan007 requested a review from nnaydenow May 26, 2026 10:23
@ilhan007 ilhan007 merged commit 4ea1e2a into main May 26, 2026
11 checks passed
@ilhan007 ilhan007 deleted the ci/skip-deploy-preview-for-forks branch May 26, 2026 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants