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: setup universal browser and node builds #234

Merged
merged 1 commit into from
Jun 25, 2020

Conversation

TrySound
Copy link
Contributor

In this diff I fixed es modules distribution for bundlers.

Before we had two problems

  • bundlers use "module" field to find es module build
  • bundlers cannot use provided es module with node target

In this diff I addressed both of them. Now "main" and "module" fields
points to "node" specific versions. "browser" field provides aliases
with browser compatible builds.

All modern browsers support both "module" and "browser" fields.

In this diff I fixed es modules distribution for bundlers.

Before we had two problems
- bundlers use "module" field to find es module build
- bundlers cannot use provided es module with node target

In this diff I addressed both of them. Now "main" and "module" fields
points to "node" specific versions. "browser" field provides aliases
with browser compatible builds.

All modern browsers support both "module" and "browser" fields.
@harttle harttle merged commit 97e7983 into harttle:master Jun 25, 2020
@harttle
Copy link
Owner

harttle commented Jun 25, 2020

@all-contributors please add TrySound for code

@allcontributors
Copy link
Contributor

@harttle

I've put up a pull request to add @TrySound! 🎉

@harttle
Copy link
Owner

harttle commented Jun 25, 2020

🎉 This PR is included in version 9.14.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@TrySound
Copy link
Contributor Author

It was fast, thanks!

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

Successfully merging this pull request may close these issues.

None yet

2 participants