Skip to content

ci: deploy virtool.ca via Cloudflare Workers#68

Merged
igboyes merged 1 commit intomainfrom
cf-workers
May 5, 2026
Merged

ci: deploy virtool.ca via Cloudflare Workers#68
igboyes merged 1 commit intomainfrom
cf-workers

Conversation

@igboyes
Copy link
Copy Markdown
Member

@igboyes igboyes commented May 5, 2026

Summary

  • Adds wrangler.jsonc configuration to deploy the static Astro site as a Cloudflare Worker with proper asset handling and trailing-slash normalization
  • Adds wrangler as a dev dependency and introduces deploy and preview npm scripts that build then run via Wrangler (original Astro preview preserved as preview:astro)

Test plan

  • Run npm run build to confirm the site builds without errors
  • Run npm run preview and verify the site is served correctly via Wrangler locally
  • Run npm run deploy and confirm the site deploys to Cloudflare Workers
  • Verify 404 pages and trailing-slash redirects work as expected on the deployed worker

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 5, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
virtool-ca 2bccb80 May 05 2026, 08:36 PM

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request integrates Cloudflare Wrangler into the project to facilitate deployment and local previewing of built assets. Key changes include adding wrangler to the development dependencies, updating the preview script, and adding a new deploy command in package.json. Additionally, a wrangler.jsonc configuration file has been introduced to define the project's asset handling and compatibility settings. There were no review comments provided, and I have no further feedback on these changes.

@igboyes igboyes merged commit dad6d99 into main May 5, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant