Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove polyfills and re-exports for built-ins on node >= 18 #60

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

Conversation

leefreemanxyz
Copy link

This is a fresh attempt at #42. From what I can ascertain, the blob package is no longer required and doesn't need importing in other packages as it should be provided by the environment, likewise for TextEncode/TextDecoder. Readable Stream can be left as is until it is no longer experimental in Remix's supported node versions.

Copy link

changeset-bot bot commented Mar 7, 2024

🦋 Changeset detected

Latest commit: a7710bd

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@remix-run/web-fetch Major
@remix-run/web-file Major
@remix-run/web-form-data Major
@remix-run/web-stream Major

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

@leefreemanxyz leefreemanxyz force-pushed the feat/remove-web-encodings-dependency branch from fae12ff to a7710bd Compare March 25, 2024 19:03
@younes200
Copy link

Can this be merged to resolve the issue with ReadableStream?
remix-run/remix#9389
Thank you

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants