[Workers] Update GitHub Actions example to use wrangler-action@v4 and note build step - #33510
Open
Guardiansoftiktok wants to merge 1 commit into
Conversation
Guardiansoftiktok
requested review from
a team,
GregBrimble,
MattieTK,
ericclemmons,
irvinebroque,
vy-ton and
yomna-shousha
as code owners
September 17, 2026 18:56
Contributor
Review✅ No issues found in commit Code ReviewThis code review is in beta and may not always be helpful — use your judgment. No code review issues found. ConventionsNo convention issues found. Style Guide ReviewNo style-guide issues found. CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
Contributor
Review coverage🟡 1 ownership area needs approval. Suggested reviewers: @dinasaur404 @irvinebroque
CODEOWNERS mappings for displayed areas (1)
|
|
Commit
Noong Biy, Set 18, 2026, 2:58 AM, isinulat ni cloudflare-docs-bot[bot] <
***@***.***> ang:
… *cloudflare-docs-bot[bot]* left a comment
(cloudflare/cloudflare-docs#33510)
<#33510 (comment)>
Review coverage
🟡 *1 ownership area needs approval.*
Suggested reviewers: @korinne <https://github.com/korinne> @yomna-shousha
<https://github.com/yomna-shousha>
Needs approval Eligible approvers Suggested
*workers* @aninibread, @ericclemmons, @GregBrimble, @irvinebroque,
@MattieTK, @vy-ton, @yomna-shousha, @cloudflare/product-owners korinne,
yomna-shousha CODEOWNERS mappings for displayed areas (1)
Pattern Owners
/src/content/docs/workers/ci-cd/ @aninibread, @ericclemmons, @GregBrimble,
@irvinebroque, @MattieTK, @vy-ton, @yomna-shousha,
@cloudflare/product-owners
—
Reply to this email directly, view it on GitHub
<#33510?email_source=notifications&email_token=B7WGP6643ITRHQG3POGO2X35PQX47A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKNZRHE3DKNBVHEZ2M4TFMFZW63VKON2WE43DOJUWEZLEUVSXMZLOOSWGM33PORSXEX3DNRUWG2Y#issuecomment-5719654593>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/B7WGP62Q74IXOCEVGNTZDST5PQX47AVCNFSNUABFKJSXA33TNF2G64TZHMZDSMRWG4ZTIMRUHNEXG43VMU5TKNBZGE2DSNZXGEZKC5QC>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
|
Commit
Noong Biy, Set 18, 2026, 3:11 AM, isinulat ni Nyree Gloriani <
***@***.***> ang:
… Commit
Noong Biy, Set 18, 2026, 2:58 AM, isinulat ni cloudflare-docs-bot[bot] <
***@***.***> ang:
> *cloudflare-docs-bot[bot]* left a comment
> (cloudflare/cloudflare-docs#33510)
> <#33510 (comment)>
> Review coverage
>
> 🟡 *1 ownership area needs approval.*
>
> Suggested reviewers: @korinne <https://github.com/korinne> @yomna-shousha
> <https://github.com/yomna-shousha>
> Needs approval Eligible approvers Suggested
> *workers* @aninibread, @ericclemmons, @GregBrimble, @irvinebroque,
> @MattieTK, @vy-ton, @yomna-shousha, @cloudflare/product-owners korinne,
> yomna-shousha CODEOWNERS mappings for displayed areas (1)
> Pattern Owners
> /src/content/docs/workers/ci-cd/ @aninibread, @ericclemmons, @GregBrimble,
> @irvinebroque, @MattieTK, @vy-ton, @yomna-shousha,
> @cloudflare/product-owners
>
> —
> Reply to this email directly, view it on GitHub
> <#33510?email_source=notifications&email_token=B7WGP6643ITRHQG3POGO2X35PQX47A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKNZRHE3DKNBVHEZ2M4TFMFZW63VKON2WE43DOJUWEZLEUVSXMZLOOSWGM33PORSXEX3DNRUWG2Y#issuecomment-5719654593>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/B7WGP62Q74IXOCEVGNTZDST5PQX47AVCNFSNUABFKJSXA33TNF2G64TZHMZDSMRWG4ZTIMRUHNEXG43VMU5TKNBZGE2DSNZXGEZKC5QC>
> .
> You are receiving this because you are subscribed to this thread.Message
> ID: ***@***.***>
>
|
… note build step - Bump cloudflare/wrangler-action@v3 to @v4 (matches current default). - Add commented-out 'npm ci && npm run build' block matching the style used on the Pages direct-upload CI page, so users deploying Workers built with a bundler or framework know where the build step goes. - Rename 'Build & Deploy Worker' step to 'Deploy Worker', since the build step is now shown separately. Reported in cloudflare/wrangler-action#408.
Guardiansoftiktok
force-pushed
the
docs/wrangler-action-v4-github-actions-example
branch
from
September 19, 2026 13:35
8dc925f to
2df0a24
Compare
Author
|
Rebased onto
Happy to trim further if the smaller scope after #33536 makes any of this not worth carrying. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Updates the GitHub Actions example on the Workers CI/CD page so that:
cloudflare/wrangler-action@v3→@v4— matches the current default of the action (Wrangler v4). Users copying the example today land on the older major version.# - name: Build/# run: npm ci && npm run build. This mirrors the pattern already used on the Pages direct-upload CI page, so users deploying Workers built with a bundler or framework (Tanstack Start, Next-on-Workers, Astro, etc.) see where to hook the build in. Without it, users have been hitting runtime errors likeentry-point file at "..." was not foundbecause the example doesn't mention a build.Build & Deploy Worker→Deploy Worker— the old name implied a build was happening inside the wrangler-action step, which was misleading.Reported in cloudflare/wrangler-action#408.
Test plan