Skip to content

Conversation

petebacondarwin
Copy link
Contributor

This is necessary because some package managers will incorrectly merge peer dependencies with actual dependencies (often ignoring peer dependency constraints).


  • Tests
    • Tests included
    • Tests not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: internal fix
  • Wrangler V3 Backport
    • Wrangler PR:
    • Not necessary because: vite-plugin not backported

@petebacondarwin petebacondarwin requested review from a team as code owners October 2, 2025 10:34
Copy link

changeset-bot bot commented Oct 2, 2025

🦋 Changeset detected

Latest commit: f09b8cd

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

Copy link

pkg-pr-new bot commented Oct 2, 2025

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

wrangler

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

commit: f09b8cd

Copy link
Contributor

@jamesopstad jamesopstad left a comment

Choose a reason for hiding this comment

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

Thanks Pete!

@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Oct 3, 2025
@petebacondarwin petebacondarwin force-pushed the pbd/vite-plugin/check-wrangler-version branch from 2eb2f60 to aee4e9c Compare October 3, 2025 15:18
@petebacondarwin petebacondarwin force-pushed the pbd/vite-plugin/check-wrangler-version branch 3 times, most recently from 44c6011 to 25eb294 Compare October 7, 2025 18:35
@petebacondarwin petebacondarwin force-pushed the pbd/vite-plugin/check-wrangler-version branch 2 times, most recently from 6ec58f6 to b8d8b65 Compare October 16, 2025 10:26
petebacondarwin and others added 8 commits October 17, 2025 13:11
This change delays confusing "missing export" failures to allow us to run a more user friendly version skew check, for when the user installs a version of Wrangler that breaks the plugin
…being used internally is correct

This is necessary because some package managers will incorrectly merge peer dependencies with actual dependencies (often ignoring peer dependency constraints).
@petebacondarwin petebacondarwin force-pushed the pbd/vite-plugin/check-wrangler-version branch from b8d8b65 to f09b8cd Compare October 17, 2025 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Approved

Development

Successfully merging this pull request may close these issues.

3 participants