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
Add ESM build to react-is #13272
Comments
You can't make it work by changing the source — you'll want to change the build scripts ( |
FYI I'm working on this. |
Could we change this issue (or should I open a new issue) to apply this to all react packages? Rollup has trouble when dependencies use React like it has an ESM export but it actually doesn't :-( Ref: reach/reach-ui#74 |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contribution. |
Not stale. |
This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. We are sorry that we haven't been able to prioritize it yet. If you have any new additional information, please include it with your comment! |
Closing this issue after a prolonged period of inactivity. If this issue is still present in the latest release, please create a new issue with up-to-date information. Thank you! |
Not stale. |
Do you want to request a feature or report a bug?
Feature
What is the current behavior?
Unable to import individual modules from the package.
What is the expected behavior?
Context
This issue arose when trying to to make
react-redux
work withReact.forwardRef
(see this PR), but evidently,react-is
doesn't allow import/export ESM to enable tree shaking capabilities inreact-redux
(that is my understanding).This issue is following this comment.
I tried solving this (and totally failed) in #13250 before I realised anything about ESM. But this is now over my head, so I'm creating an issue now and see how that follows.
The text was updated successfully, but these errors were encountered: