Skip to content

Skip deploy previews for fork PRs#67

Merged
southpolesteve merged 1 commit intomainfrom
fix/skip-deploy-for-forks
Feb 25, 2026
Merged

Skip deploy previews for fork PRs#67
southpolesteve merged 1 commit intomainfrom
fix/skip-deploy-for-forks

Conversation

@southpolesteve
Copy link
Copy Markdown
Collaborator

Summary

  • Fork PRs don't have access to repository secrets (CLOUDFLARE_API_TOKEN, CLOUDFLARE_ACCOUNT_ID), causing all deploy jobs to fail with red X's whenever an external contributor opens a PR
  • Adds github.event.pull_request.head.repo.full_name == github.repository condition to the deploy preview step, smoke test preview step, and comment job
  • Example builds still run for fork PRs (validates the code compiles), only the deploy/smoke-test/comment steps are skipped

This matches the workers-sdk pattern where preview deploys are only run for internal branches.

Fork PRs don't have access to repository secrets (CLOUDFLARE_API_TOKEN,
CLOUDFLARE_ACCOUNT_ID), so deploy/smoke-test/comment steps always fail.
Add a fork detection condition to skip these steps gracefully. The build
steps still run, so example builds are still validated for fork PRs.
@github-actions
Copy link
Copy Markdown

Example Preview Production Original
app-router-cloudflare preview production
pages-router-cloudflare preview production
app-router-playground preview production original
realworld-api-rest preview production
nextra-docs-template preview production
benchmarks preview production
hackernews preview production original

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Feb 25, 2026

Open in StackBlitz

npm i https://pkg.pr.new/cloudflare/vinext@67

commit: d951af6

@southpolesteve southpolesteve merged commit bc75de3 into main Feb 25, 2026
57 of 64 checks passed
@southpolesteve southpolesteve deleted the fix/skip-deploy-for-forks branch February 25, 2026 19:47
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.

1 participant