-
Notifications
You must be signed in to change notification settings - Fork 9.9k
update containers local dev socket configuration docs #23963
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
Conversation
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
| You may prefer to set up your own code watchers and reloading mechanisms, or mount a local directory | ||
| into the local container images to sync code changes. This can be done, but there is no built-in | ||
| mechanism for doing so in Wrangler, and best-practices will depend on the languages and frameworks | ||
| mechanism for doing so here, and best-practices will depend on the languages and frameworks |
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.
Can we be more specific where it says here?
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.
i've removed here entirely if that's ok, i think it should be clear from context. Trying to remove specific references to wrangler in this PR.
4d7e29d to
9291c10
Compare
* updates to socket configuration docs * update wording
ref cloudflare/workers-sdk#10061