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

feat: expose port and address on (Unstable)DevWorker #1913

Merged
merged 1 commit into from Sep 23, 2022

Conversation

threepointone
Copy link
Contributor

when using unstable_dev(), I think we want to expose the port/address that the server has started on. The usecase is when trying to connect to the server without calling .fetch() (example: when making a websocket connection).

when using `unstable_dev()`, I think we want to expose the port/address that the server has started on. The usecase is when trying to connect to the server _without_ calling `.fetch()` (example: when making a websocket connection).
@changeset-bot
Copy link

changeset-bot bot commented Sep 23, 2022

🦋 Changeset detected

Latest commit: 5575d64

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/3111448758/npm-package-wrangler-1913

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

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

Or you can use npx with this latest build directly:

npx https://prerelease-registry.developers.workers.dev/runs/3111448758/npm-package-wrangler-1913 dev path/to/script.js

@codecov
Copy link

codecov bot commented Sep 23, 2022

Codecov Report

Merging #1913 (5575d64) into main (b7976b6) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1913      +/-   ##
==========================================
+ Coverage   77.48%   77.50%   +0.01%     
==========================================
  Files         105      105              
  Lines        7480     7480              
  Branches     1974     1974              
==========================================
+ Hits         5796     5797       +1     
+ Misses       1684     1683       -1     
Impacted Files Coverage Δ
packages/wrangler/src/api/dev.ts 69.23% <100.00%> (ø)
...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.

fair

@threepointone
Copy link
Contributor Author

thanks bruh

@threepointone threepointone merged commit 9f7cc5a into main Sep 23, 2022
@threepointone threepointone deleted the dev-expose-port-address branch September 23, 2022 08:53
@github-actions github-actions bot mentioned this pull request Sep 23, 2022
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.

None yet

2 participants