-
Notifications
You must be signed in to change notification settings - Fork 803
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Version Packages #8386
base: main
Are you sure you want to change the base?
Version Packages #8386
Conversation
A wrangler prerelease is available for testing. You can install this latest build in your project with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13763484836/npm-package-wrangler-8386 You can reference the automatically updated head of this PR with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/8386/npm-package-wrangler-8386 Or you can use npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13763484836/npm-package-wrangler-8386 dev path/to/script.js Additional artifacts:cloudflare-workers-bindings-extension: wget https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13763484836/npm-package-cloudflare-workers-bindings-extension-8386 -O ./cloudflare-workers-bindings-extension.0.0.0-vac1cd00a1.vsix && code --install-extension ./cloudflare-workers-bindings-extension.0.0.0-vac1cd00a1.vsix create-cloudflare: npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13763484836/npm-package-create-cloudflare-8386 --no-auto-update @cloudflare/kv-asset-handler: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13763484836/npm-package-cloudflare-kv-asset-handler-8386 miniflare: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13763484836/npm-package-miniflare-8386 @cloudflare/pages-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13763484836/npm-package-cloudflare-pages-shared-8386 @cloudflare/unenv-preset: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13763484836/npm-package-cloudflare-unenv-preset-8386 @cloudflare/vite-plugin: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13763484836/npm-package-cloudflare-vite-plugin-8386 @cloudflare/vitest-pool-workers: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13763484836/npm-package-cloudflare-vitest-pool-workers-8386 @cloudflare/workers-editor-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13763484836/npm-package-cloudflare-workers-editor-shared-8386 @cloudflare/workers-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13763484836/npm-package-cloudflare-workers-shared-8386 @cloudflare/workflows-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13763484836/npm-package-cloudflare-workflows-shared-8386 Note that these links will no longer work once the GitHub Actions artifact expires.
Please ensure constraints are pinned, and |
3526891
to
f95923d
Compare
60ffb50
to
2c4a66b
Compare
@@ -1,5 +1,16 @@ | |||
# @cloudflare/vite-plugin | |||
|
|||
## 1.0.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is this being marked as a major? I couldn't figure out which changeset marked it so @penalosa @jamesopstad
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, this isn't right. It's not obvious to me what's going on here. #8273 changed Wrangler from being a dev dependency to a dependency so maybe that has something to do with it or it could be related to unenv somehow.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe a bug in changesets? changesets/changesets#1011
I suspect it is that we released a major bump to unenv-preset (2.0.0) and vite-plugin has a workspace dependency on that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For now I think we might be able to just manually update this PR before merging...?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, so looks like this is documented here - https://github.com/changesets/changesets/blob/main/docs/decisions.md#the-versioning-of-peer-dependencies. It seems that if a peer dependency has a minor version bump then the package with the peer dependency is given a major version bump.
We definitely don't want this so I think we have two options:
- Remove the peer dependency on Wrangler altogether (this is only there to try and enforce that the version used for deployment uses the redirected config)
- Use the version range we had before Fix workers ai binding for cf vite plugin #8273 for the peer dependency rather than
workspace:*
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The latter.
- Updated dependencies [[`8d6d722`](https://github.com/cloudflare/workers-sdk/commit/8d6d7224bcebe04691478e2c5261c00992a1747a), [`e3efd68`](https://github.com/cloudflare/workers-sdk/commit/e3efd68e3989815f6935fa4315e0aa23aaac11c9), [`53e6323`](https://github.com/cloudflare/workers-sdk/commit/53e63233c5b9bb786af3daea63c10ffe60a5d881), [`4d9d9e6`](https://github.com/cloudflare/workers-sdk/commit/4d9d9e6c830b32a0e9948ace32e20a1cdac3a53b)]: | ||
- [email protected] | ||
- [email protected] | ||
- @cloudflare/[email protected] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm sus
ddb7c3e
to
316ed9a
Compare
316ed9a
to
4078b30
Compare
This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
[email protected]
Minor Changes
3536670
Thanks @emily-shen! - feat: update nitro config in nuxt templates to use generated wrangler configPatch Changes
#8372
3536670
Thanks @emily-shen! - Graduate Angular, Gatsby, Vue and Nuxt templates (Workers + Assets) from experimentalYou no longer need to pass in the --experimental flag to access these templates.
[email protected]
Minor Changes
53e6323
Thanks @GregBrimble! - Parse and apply metafiles (_headers
and_redirects
) inwrangler dev
for Workers Assets@cloudflare/[email protected]
Minor Changes
53e6323
Thanks @GregBrimble! - Parse and apply metafiles (_headers
and_redirects
) inwrangler dev
for Workers AssetsPatch Changes
53e6323
,08b8c46
,aba0e9c
]:@cloudflare/[email protected]
Minor Changes
#8390
53e6323
Thanks @GregBrimble! - Parse and apply metafiles (_headers
and_redirects
) inwrangler dev
for Workers Assets#8373
08b8c46
Thanks @WalshyDev! - AddCF-Cache-Status
to Workers Assets responses to indicate if we returned a cached asset or not. This will also populate zone cache analytics and Logpush logs.#8279
aba0e9c
Thanks @GregBrimble! - Add support for custom headers and redirects in asset-worker[email protected]
Minor Changes
53e6323
Thanks @GregBrimble! - Parse and apply metafiles (_headers
and_redirects
) inwrangler dev
for Workers AssetsPatch Changes
#8383
8d6d722
Thanks @matthewdavidrodgers! - Make kv bulk put --local respect base64:trueThe bulk put api has an optional "base64" boolean property for each key.
Before storing the key, the value should be decoded from base64.
For real (remote) kv, this is handled by the rest api. For local kv, it
seems the base64 field was ignored, meaning encoded base64 content was
stored locally rather than the raw values.
To fix, we need to decode each value before putting to the local
miniflare namespace when base64 is true.
#8273
e3efd68
Thanks @penalosa! - Support AI, Vectorize, and Images bindings when using@cloudflare/vite-plugin
#8392
4d9d9e6
Thanks @jahands! - fix: retry zone and route lookup API callsIn rare cases, looking up Zone or Route API calls may fail due to transient errors. This change improves the reliability of
wrangler deploy
when these errors occur.Also fixes a rare issue where concurrent API requests may fail without correctly throwing an error which may cause a deployment to incorrectly appear successful.
Updated dependencies [
53e6323
]:@cloudflare/[email protected]
Patch Changes
#8273
e3efd68
Thanks @penalosa! - Support AI, Vectorize, and Images bindings when using@cloudflare/vite-plugin
Updated dependencies [
8d6d722
,e3efd68
,53e6323
,4d9d9e6
]:@cloudflare/[email protected]
Patch Changes
8d6d722
,e3efd68
,53e6323
,4d9d9e6
]: