-
Notifications
You must be signed in to change notification settings - Fork 25k
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
build: fix secondary entry point es5 output and core Rx references #14820
Conversation
Secondary entry points (testing, static, etc) are rolled up into a single ESM/ES2015 file, then downleveled to ESM/ES2015. This downleveling was not working and was producing ESM/ES2015. Also, the @angular/core package's .babelrc file was missing reference to Rx Observable which broke the UMD bundle. Fixes angular#14730
"rxjs/Observer": "Rx", | ||
"rxjs/Subscription": "Rx", | ||
|
||
"rxjs/symbol/observable": "Rx.Symbol.observable", |
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.
I'm just not sure this is correct. I created an issue on rxjs repo ReactiveX/rxjs#2415 but they haven't answered anything >_<
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.
Yes, looks like you're right. Leaving in atm since this is going back to what previous rollup.config.js was. But it looks like it will be broken until the issue you filed gets fixed.
…ngular#14820) Secondary entry points (testing, static, etc) are rolled up into a single ESM/ES2015 file, then downleveled to ESM/ES2015. This downleveling was not working and was producing ESM/ES2015. Also, the @angular/core package's .babelrc file was missing reference to Rx Observable which broke the UMD bundle. Fixes angular#14730
…ngular#14820) Secondary entry points (testing, static, etc) are rolled up into a single ESM/ES2015 file, then downleveled to ESM/ES2015. This downleveling was not working and was producing ESM/ES2015. Also, the @angular/core package's .babelrc file was missing reference to Rx Observable which broke the UMD bundle. Fixes angular#14730
…ngular#14820) Secondary entry points (testing, static, etc) are rolled up into a single ESM/ES2015 file, then downleveled to ESM/ES2015. This downleveling was not working and was producing ESM/ES2015. Also, the @angular/core package's .babelrc file was missing reference to Rx Observable which broke the UMD bundle. Fixes angular#14730
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Secondary entry points (testing, static, etc) are rolled up into a
single ESM/ES2015 file, then downleveled to ESM/ES2015. This downleveling
was not working and was producing ESM/ES2015. Also, the @angular/core
package's .babelrc file was missing reference to Rx Observable which
broke the UMD bundle.
Fixes #14730