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

Fixed commonjs build #122

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

Fixed commonjs build #122

wants to merge 5 commits into from

Conversation

oskardudycz
Copy link
Collaborator

Removed async import of the node:stream/web. It's redundant as in the modern browsers and node.js it should be available in globalThis object.

@oskardudycz oskardudycz added bug Something isn't working CI compatibility labels Oct 11, 2024
@oskardudycz oskardudycz force-pushed the fixed_commonjs_build branch 4 times, most recently from ee8cd5d to 8313646 Compare October 11, 2024 19:04
Removed async import of the node:stream/web. It's redundant as in the
modern browsers it should be available in globalThis object
This will need to be changed somehow in the end, but for now this code
is not used
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CI compatibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant