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

Bump unjs ecosystem #1508

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Bump unjs ecosystem #1508

wants to merge 2 commits into from

Conversation

lishaduck
Copy link
Contributor

@lishaduck lishaduck commented Mar 8, 2025

Overview

Following https://antfu.me/posts/move-on-to-esm-only, the unjs ecosystem is now ESM only.
This means that we need to extend our range of supported package versions.

Notably, I didn't upgrade ohash, because v2 removes murmur in favor of digest. (Which is ironic, because that's why I started this PR).

Other notes:

Manual Testing

Shouldn't be needed, these are all just ESM-only migrations.

Related Issue

None

Following https://antfu.me/posts/move-on-to-esm-only,
the unjs ecosystem is now ESM only.
This means that we need to extend our range of supported package versions.
Copy link

netlify bot commented Mar 8, 2025

Deploy Preview for creative-fairy-df92c4 ready!

Name Link
🔨 Latest commit 8438394
🔍 Latest deploy log https://app.netlify.com/sites/creative-fairy-df92c4/deploys/67cd024221ae910008c2d42f
😎 Deploy Preview https://deploy-preview-1508--creative-fairy-df92c4.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@lishaduck
Copy link
Contributor Author

Shouldn't be needed, these are all just ESM-only migrations.

I take that back, nypm now supports Deno, and the way we're using nypm makes that a breaking change.
I gave a shot at updating, but Deno's so different I don't know how we want to handle it.

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.

1 participant