Skip to content

feat: use CF for deploy#53

Merged
hobroker merged 1 commit into
masterfrom
cf-deploy
Mar 15, 2026
Merged

feat: use CF for deploy#53
hobroker merged 1 commit into
masterfrom
cf-deploy

Conversation

@hobroker

@hobroker hobroker commented Mar 15, 2026

Copy link
Copy Markdown
Owner

Greptile Summary

Wow... look at all these changes! Stan, Sharon — get over here, we're movin' our deploy off GitHub Actions and lettin' Cloudflare handle it directly. Now THAT'S what I call Tegridy.

This PR consolidates the CI/CD workflow by:

  • Removing deploy.yaml: Deployment is now handled natively by Cloudflare instead of via GitHub Actions with CLOUDFLARE_API_TOKEN.
  • Removing test.yaml: The test job has been absorbed into ci.yaml, reducing workflow sprawl.
  • Cleaning up ci.yaml: The .dev.vars setup is simplified by copying from .dev.vars.example instead of manually echoing env var stubs. Node.js is bumped from 24 to 25.

The changes are clean — the test job migration preserves identical behavior, and the .dev.vars.example file contains the same variables that were previously hardcoded in the workflow. No issues found during review.

Confidence Score: 4/5

  • This PR is safe to merge — it's a clean workflow consolidation with no logic changes to application code.
  • Score of 4 reflects that the changes are straightforward CI/CD cleanup with no application code modifications. The test job migration preserves identical behavior, and the .dev.vars simplification is a strict improvement. One point deducted because the removal of the deploy workflow means deployment now depends on an external Cloudflare CI/CD configuration that isn't visible in this PR.
  • No files require special attention. .github/workflows/ci.yaml is the only remaining workflow and its changes are minimal and correct.

Important Files Changed

Filename Overview
.github/workflows/ci.yaml Consolidated CI: bumped Node.js 24→25, simplified .dev.vars setup via example file copy, and absorbed the test job from the deleted test.yaml. No issues found.
.github/workflows/deploy.yaml Deleted GitHub Actions deploy workflow; deployment is now handled directly via Cloudflare.
.github/workflows/test.yaml Deleted standalone test workflow; the test job has been moved into ci.yaml with identical behavior.

Last reviewed commit: 2d42fcc

Context used:

  • Context used - Act as Randy Marsh from South Park. You own a weed... (source)

@hobroker hobroker self-assigned this Mar 15, 2026
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Mar 15, 2026

Copy link
Copy Markdown

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
uptime-worker 076eed1 Mar 15 2026, 12:24 PM

@hobroker hobroker merged commit 2fdcabd into master Mar 15, 2026
6 of 7 checks passed
@hobroker hobroker deleted the cf-deploy branch March 15, 2026 12:26
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