-
Notifications
You must be signed in to change notification settings - Fork 522
Add support for es2015 entry point field in rollup #429
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 support for es2015 entry point field in rollup #429
Conversation
|
/cc @alexeagle |
|
@gregmagolan @alexeagle is out of office for the next a couple of days. I can review the PR. I'll ping you on Slack to get a better context. |
alexeagle
left a comment
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.
Since you have WIP on here, I assume you don't want to merge this until you get a PR merged to the upstream? Probably a wise idea to make sure the next release of rollup-plugin-node-resolve will have what we need
|
@alexeagle Yes. Will have some time tomorrow morning to make a PR to the node resolve plugin |
|
PR submitted to rollup-plugun-node-resolve: rollup/rollup-plugin-node-resolve#186 |
|
Looks like this change in rollup-plugin-node-resolve was already in the works which will allow for handling |
|
rollup/rollup-plugin-node-resolve#186 has still not landed. I made a custom dist based on rollup/rollup-plugin-node-resolve#186 which has been rebased to 4.0.0. Will go with that. After #186 lands and makes it into the next rollup release we can update to that version. Its all behind the scenes so it shouldn't break anyone. |
These already moved to rules_nodejs Closes bazel-contrib#429 PiperOrigin-RevId: 241422158
These already moved to rules_nodejs Closes bazel-contrib#429 PiperOrigin-RevId: 241422158
Update to a version of rollup-plugin-node-resolve that supports
es2015entry points with fallback tomoduleandmain: https://github.com/gregmagolan/rollup-plugin-node-resolve/commit/7a33dc249b69497aff3380a37a359158167b4801.Will put a PR up for rollup-plugin-node-resolve to land this in their repo.
Needed when rolling up rxjs here: angular/angular#27220