-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Fix regression where polyfills where build into into extensions. #1681
Conversation
4818a81
to
d9f5cbc
Compare
@@ -14,6 +14,7 @@ | |||
* limitations under the License. | |||
*/ | |||
|
|||
import './polyfills'; |
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.
Can we remove the import in timers.js
?
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.
We really need a better solution. Basically you need to put it everywhere that uses promises and has an immediate side effect.
d9f5cbc
to
6c9efcc
Compare
PTAL |
// Many files include the polyfills, but we only want to deliver them | ||
// once. Since all files automatically wait for the main binary to load | ||
// this works fine. | ||
if (options.includePolyfills) { |
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 think the conditional is backwards?
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.
It isn't.
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.
There are two files that resolve to the module name. One is empty. If we should include polyfills, it removes the empty from the set, otherwise it removes the actual file.
@cramforce tested, LGTM |
Fix regression where polyfills where build into into extensions.
No description provided.