Skip to content
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

chore: error if d1 bindings used with no-bundle #1955

Merged
merged 2 commits into from Sep 29, 2022
Merged

Conversation

cameron-robey
Copy link
Contributor

While in beta, you cannot use D1 bindings without bundling your worker as these are added in through a facade which gets bypassed when using the no-bundle option.

This will be removed once the bindings are no longer needing to be added using a shim.

Closes #1954

@changeset-bot
Copy link

changeset-bot bot commented Sep 29, 2022

🦋 Changeset detected

Latest commit: 6e46562

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

This PR includes changesets to release 1 package
Name Type
wrangler Patch

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

@github-actions
Copy link
Contributor

A wrangler prerelease is available for testing. You can install this latest build in your project with:

npm install --save-dev https://prerelease-registry.developers.workers.dev/runs/3151349412/npm-package-wrangler-1955

You can reference the automatically updated head of this PR with:

npm install --save-dev https://prerelease-registry.developers.workers.dev/prs/1955/npm-package-wrangler-1955

Or you can use npx with this latest build directly:

npx https://prerelease-registry.developers.workers.dev/runs/3151349412/npm-package-wrangler-1955 dev path/to/script.js
Additional artifacts:
npm install https://prerelease-registry.developers.workers.dev/runs/3151349412/npm-package-cloudflare-pages-shared-1955

Copy link
Member

@GregBrimble GregBrimble left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link

codecov bot commented Sep 29, 2022

Codecov Report

Merging #1955 (6e46562) into main (27784fb) will increase coverage by 0.00%.
The diff coverage is 66.66%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1955   +/-   ##
=======================================
  Coverage   74.96%   74.96%           
=======================================
  Files         116      116           
  Lines        7952     7958    +6     
  Branches     2074     2078    +4     
=======================================
+ Hits         5961     5966    +5     
- Misses       1991     1992    +1     
Impacted Files Coverage Δ
packages/wrangler/src/publish.ts 91.26% <66.66%> (-0.49%) ⬇️
...ackages/wrangler/src/__tests__/helpers/mock-bin.ts 100.00% <0.00%> (+5.26%) ⬆️

Copy link
Contributor

@rozenmd rozenmd left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 BUG: Cannot access D1 bindings if no_bundle is set
3 participants