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(cloudflare): use workerd implementation of node:dns #376

Merged
merged 3 commits into from
Dec 16, 2024
Merged

Conversation

vicb
Copy link
Contributor

@vicb vicb commented Dec 16, 2024

@vicb vicb requested review from a team and pi0 as code owners December 16, 2024 13:31
@vicb vicb requested a review from anonrig December 16, 2024 13:32
@vicb vicb requested a review from anonrig December 16, 2024 14:18
@pi0 pi0 merged commit 7e05819 into main Dec 16, 2024
2 checks passed
@vicb
Copy link
Contributor Author

vicb commented Dec 16, 2024

Thanks for the reviews!

@vicb vicb deleted the node/dns branch December 16, 2024 14:59
renovate bot added a commit to zenobi-us/zenobi-us that referenced this pull request Jan 9, 2025
This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [wrangler](https://redirect.github.com/cloudflare/workers-sdk)
([source](https://redirect.github.com/cloudflare/workers-sdk/tree/HEAD/packages/wrangler))
| [`3.99.0` ->
`3.100.0`](https://renovatebot.com/diffs/npm/wrangler/3.99.0/3.100.0) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/wrangler/3.100.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/wrangler/3.100.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/wrangler/3.99.0/3.100.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/wrangler/3.99.0/3.100.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>cloudflare/workers-sdk (wrangler)</summary>

###
[`v3.100.0`](https://redirect.github.com/cloudflare/workers-sdk/blob/HEAD/packages/wrangler/CHANGELOG.md#31000)

[Compare
Source](https://redirect.github.com/cloudflare/workers-sdk/compare/[email protected]@3.100.0)

##### Minor Changes

-
[#&#8203;7604](https://redirect.github.com/cloudflare/workers-sdk/pull/7604)
[`6c2f173`](https://redirect.github.com/cloudflare/workers-sdk/commit/6c2f17341037962bdf675e7008a4d91059465e16)
Thanks
[@&#8203;CarmenPopoviciu](https://redirect.github.com/CarmenPopoviciu)!
- feat: Capture Workers with static assets in the telemetry data

We want to measure accurately what this number of Workers + Assets
projects running in remote mode is, as this number will be a very
helpful data point down the road, when more decisions around remote mode
will have to be taken.

These changes add this kind of insight to our telemetry data, by
capturing whether the command running is in the context of a Workers +
Assets project.

N.B. With these changes in place we will be capturing the Workers +
Assets context for all commands, not just wrangler dev --remote.

##### Patch Changes

-
[#&#8203;7581](https://redirect.github.com/cloudflare/workers-sdk/pull/7581)
[`cac7fa6`](https://redirect.github.com/cloudflare/workers-sdk/commit/cac7fa6160ecc70d8f188de1f494a07c0e1e9626)
Thanks [@&#8203;vicb](https://redirect.github.com/vicb)! -
chore(wrangler): update unenv dependency version

    unenv now uses the workerd implementation on node:dns
See the
[unjs/unenv#376](https://redirect.github.com/unjs/unenv/pull/376)

-
[#&#8203;7625](https://redirect.github.com/cloudflare/workers-sdk/pull/7625)
[`d8fb032`](https://redirect.github.com/cloudflare/workers-sdk/commit/d8fb032ba24ac284147dc481c28ab8dbcf7a9d72)
Thanks [@&#8203;vicb](https://redirect.github.com/vicb)! -
feat(wrangler): use unenv builtin dependency resolution

Moving away from `require.resolve()` to handle unenv aliased packages.
Using the unenv builtin resolution will allow us to drop the .cjs file
from the preset
and to override the base path so that we can test the dev version of the
preset.

-
[#&#8203;7533](https://redirect.github.com/cloudflare/workers-sdk/pull/7533)
[`755a27c`](https://redirect.github.com/cloudflare/workers-sdk/commit/755a27c7a5d7f35cb5f05ab2e12af6d64ce323fb)
Thanks [@&#8203;danielgek](https://redirect.github.com/danielgek)! - Add
warning about the browser rendering not available on local

-
[#&#8203;7614](https://redirect.github.com/cloudflare/workers-sdk/pull/7614)
[`8abb43f`](https://redirect.github.com/cloudflare/workers-sdk/commit/8abb43fcdf0c506fa6268a7f07aa31b398b7daf2)
Thanks [@&#8203;vicb](https://redirect.github.com/vicb)! -
chore(wrangler): update unenv dependency version

    The updated unenv contains a fix for the module resolution,
see [unjs/unenv#378](https://redirect.github.com/unjs/unenv/pull/378).
    That bug prevented us from using unenv module resolution,
see
[#&#8203;7583](https://redirect.github.com/cloudflare/workers-sdk/pull/7583).

- Updated dependencies
\[[`b4e0af1`](https://redirect.github.com/cloudflare/workers-sdk/commit/b4e0af163548ee8cc0aefc9165f67a0f83ea94d4)]:
    -   [email protected]

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/).
View the [repository job
log](https://developer.mend.io/github/zenobi-us/zenobi-us).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOS45Mi4wIiwidXBkYXRlZEluVmVyIjoiMzkuOTIuMCIsInRhcmdldEJyYW5jaCI6Im1hc3RlciIsImxhYmVscyI6W119-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

3 participants