Changelog + docs update: Removing compressed Worker size limits#31067
Changelog + docs update: Removing compressed Worker size limits#31067yomna-shousha wants to merge 5 commits into
Conversation
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
There was a problem hiding this comment.
1 issue and 2 style items need attention before merge.
- HIGH —
changelog/workers/2026-05-22-increased-worker-size-limit.mdx:11: The "~21x more room" / "~6x more" claim compares uncompressed 64 MiB to the old compressed limits. This is misleading — the previous uncompressed limit was already 64 MB. The real change is removing the compressed ceiling, not increasing uncompressed capacity by 21x. - MEDIUM —
changelog/workers/2026-05-22-increased-worker-size-limit.mdx:6: Changelog date needs updating to the actual ship date before merge (noted in PR description). - MEDIUM — Both changed files use
txtfor command output blocks instead oftxt outputper the style guide, and include a manual# Output resembles:comment that is unnecessary when using theoutputsuffix.
Labels: documentation (applied).
| description: The compressed Worker size limits (3 MB free / 10 MB paid) have been removed. Workers now support a single 64 MiB uncompressed size limit across all plans. | ||
| products: | ||
| - workers | ||
| date: 2026-05-22 |
There was a problem hiding this comment.
The PR description notes this date needs to be updated to match when the changes ship. Please update before merging.
|
Posted a review on PR #31067 with 4 inline comments. Summary of findings:
Labels applied: |
|
/bonk rereview the changelog. the date will be updated when the change is released so you can ignore that. |
There was a problem hiding this comment.
Re-review findings:
Fixed since last review:
- Both files now correctly use
sh outputfor command output blocks instead of manual# Output resembles:comments. ✅ - Changelog date acknowledged as pending release.
Still needs attention (HIGH):
- The changelog prose and comparison table continue to misleadingly present this as a capacity increase from 3 MB / 10 MB → 64 MiB. The old
limits.mdxonproductionshows the uncompressed limit was already 64 MB. The real change is the removal of the compressed (gzip) ceiling, not a 21× increase in uncompressed Worker capacity. The body text and table should be rephrased to make that clear.
|
Done. I posted a request changes review on PR #31067. Summary:
|
|
/bonk rereview again :) |
|
Good news — all the issues from my previous review have been addressed. Here's my updated assessment: Previously flagged (HIGH) — FIXED ✅
This is a correct and clear explanation of the change. Previously flagged (MEDIUM) — FIXED ✅ Date field No new issues found. The |
|
Hey there, we've marked this pull request as stale because there's no recent activity on it. This label helps us identify PRs that might need updates (or to be closed out by our team if no longer relevant). |
…-limit.mdx Co-authored-by: Brendan Irvine-Broque <birvine-broque@cloudflare.com>
Review
Code ReviewThis code review is in beta and may not always be helpful — use your judgment. ✅ No code review issues found. Style Guide ReviewWarnings (1)
Suggestions (1)
CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
Adds a changelog entry and updates the Worker size limits documentation for the removal of compressed size limits (WC-4489).
TODO: Update the changelog date to match when changes ship.
Related PRs: