Skip to content

chore: migrate CF Pages deploys to cloudflare/wrangler-action#2736

Draft
maxffarrell wants to merge 2 commits into
huntabyte:mainfrom
maxffarrell:chore/migrate-cf-pages-wrangler-refork
Draft

chore: migrate CF Pages deploys to cloudflare/wrangler-action#2736
maxffarrell wants to merge 2 commits into
huntabyte:mainfrom
maxffarrell:chore/migrate-cf-pages-wrangler-refork

Conversation

@maxffarrell

Copy link
Copy Markdown
Contributor

AdrianGonz97/refined-cf-pages-action is deprecated. Replaces it with the official cloudflare/wrangler-action@v3 using wrangler pages deploy across the preview, svelte-4, and tailwind-3 deploy workflows.

Supersedes #2705 because the original PR head repo (maxffarrell/shadcn-svelte) is no longer in the huntabyte/shadcn-svelte fork network. This branch is republished from the reforked repo (maxffarrell/shadcn-svelte-og).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 5, 2026 03:43
@changeset-bot

changeset-bot Bot commented Jun 5, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: e689c27

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

This PR updates the Cloudflare Pages deployment workflows to use Cloudflare’s official Wrangler GitHub Action instead of a third-party Cloudflare Pages deploy action.

Changes:

  • Replace AdrianGonz97/refined-cf-pages-action@v1 with cloudflare/wrangler-action@v3 across deployment workflows
  • Switch from structured action inputs (projectName, directory, deploymentName) to wrangler pages deploy ... via command
  • Adjust branch/deployment behavior to be driven by Wrangler CLI flags

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
.github/workflows/deploy-tailwind-3.yml Migrates Tailwind 3 docs deploy step to wrangler-action with a pages deploy command
.github/workflows/deploy-svelte-4.yml Migrates Svelte 4 docs deploy step to wrangler-action, passing branch via CLI flag
.github/workflows/deploy-preview.yml Migrates preview deploy step to wrangler-action and deploys the downloaded build artifact via CLI

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

githubToken: ${{ secrets.GITHUB_TOKEN }}
projectName: shadcn-svelte-tw-3
directory: ./.svelte-kit/cloudflare
gitHubToken: ${{ secrets.GITHUB_TOKEN }}
githubToken: ${{ secrets.GITHUB_TOKEN }}
projectName: shadcn-svelte
directory: ./.svelte-kit/cloudflare
gitHubToken: ${{ secrets.GITHUB_TOKEN }}
Comment on lines +34 to +35
gitHubToken: ${{ secrets.GITHUB_TOKEN }}
command: pages deploy ${{ steps.preview-build-artifact.outputs.download-path }}/cloudflare --project-name=shadcn-svelte
gitHubToken: ${{ secrets.GITHUB_TOKEN }}
workingDirectory: sites/docs
deploymentName: Production Tailwind 3
command: pages deploy ./.svelte-kit/cloudflare --project-name=shadcn-svelte-tw-3
@maxffarrell maxffarrell marked this pull request as draft June 5, 2026 03:45
@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor
built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
shadcn-svelte ❌ Failed (View Log) e689c27

@github-actions github-actions Bot requested a deployment to Preview June 5, 2026 03:49 Abandoned
@maxffarrell

Copy link
Copy Markdown
Contributor Author

Linked issue: #2737

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