-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Node built-in modules support for Deno adapter #7288
Conversation
🦋 Changeset detectedLatest commit: b46ee61 The changes in this PR will be included in the next version bump. 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 |
6bfe244
to
044f9d6
Compare
7ee3d9c
to
b46ee61
Compare
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.
This looks like a great change and the code is straightforward!
I'm definitely on board with this but would love another review before merging.
Yeah, that sounds good. |
I would have liked to include an npm library that uses a supported node built-in module, but couldn't find one that's appropriate for CI. |
Changes
It seems to be a use case that's receiving more and more interest. Deno can leverage built-in node modules, but it's blocked by an overly restrictive configuration set by the adapter.
Testing
nodecompat.astro
Docs
Not a user facing change, though a note somewhere could be useful.