Skip to content

Add support for new placement mode#10937

Merged
dario-piotrowicz merged 1 commit intocloudflare:mainfrom
ReppCodes:arepp/placement
Dec 3, 2025
Merged

Add support for new placement mode#10937
dario-piotrowicz merged 1 commit intocloudflare:mainfrom
ReppCodes:arepp/placement

Conversation

@ReppCodes
Copy link
Contributor

@ReppCodes ReppCodes commented Oct 9, 2025

This PR refactors the placement fields supported in config files to permit the fields expected by the new placement API.

  • Tests
    • Tests included
    • Tests not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: will be done in a followup PR prior to public launch.
  • Wrangler V3 Backport
    • Wrangler PR:
    • Not necessary because: new feature will not be supported on V3

@ReppCodes ReppCodes requested review from a team as code owners October 9, 2025 14:42
@github-project-automation github-project-automation bot moved this to Untriaged in workers-sdk Oct 9, 2025
@changeset-bot
Copy link

changeset-bot bot commented Oct 9, 2025

🦋 Changeset detected

Latest commit: 446b838

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

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

@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 9, 2025

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@10937

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@10937

miniflare

npm i https://pkg.pr.new/miniflare@10937

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@10937

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@10937

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@10937

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@10937

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@10937

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@10937

wrangler

npm i https://pkg.pr.new/wrangler@10937

commit: 446b838

@lrapoport-cf lrapoport-cf added the caretaking Priority for caretaking label Oct 10, 2025
@jamesopstad jamesopstad marked this pull request as draft October 17, 2025 09:15
@petebacondarwin petebacondarwin removed the caretaking Priority for caretaking label Oct 20, 2025
@ReppCodes ReppCodes force-pushed the arepp/placement branch 3 times, most recently from 1d62528 to c48d9f7 Compare November 3, 2025 19:26
@ReppCodes ReppCodes force-pushed the arepp/placement branch 3 times, most recently from bee5ad1 to c2da26f Compare November 13, 2025 21:24
@ReppCodes ReppCodes changed the title Draft: Add support for new placement mode Add support for new placement mode Nov 13, 2025
@ReppCodes ReppCodes marked this pull request as ready for review November 13, 2025 21:55
@ReppCodes ReppCodes force-pushed the arepp/placement branch 6 times, most recently from f0003c2 to 06c7d17 Compare November 14, 2025 18:34
Copy link
Member

@dario-piotrowicz dario-piotrowicz left a comment

Choose a reason for hiding this comment

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

Overall looks good but there are a few small things that could be improved I think 🙂

@dario-piotrowicz
Copy link
Member

Also, could you add a changeset for the workers-utils package as well? 🙏

@ReppCodes ReppCodes force-pushed the arepp/placement branch 2 times, most recently from 6e89cf4 to 630d939 Compare November 20, 2025 14:40
@Ltadrian
Copy link
Contributor

Looks good after the changes from Dario's reviews!

Approved from me ✅

@ReppCodes ReppCodes force-pushed the arepp/placement branch 2 times, most recently from ad0f589 to c0c0a12 Compare November 20, 2025 16:16
@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Dec 1, 2025
Add wrangler support for targeted placement Worker configurations.
The relevant tests have been added or updated, and type-level
enforcement of the new modes should prevent invalid formats from being
passed through.
@dario-piotrowicz dario-piotrowicz merged commit 9514c9a into cloudflare:main Dec 3, 2025
53 of 62 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Done in workers-sdk Dec 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants