Skip to content

[pull] master from angular:master#165

Merged
pull[bot] merged 2 commits intoadmariner:masterfrom
angular:master
Sep 27, 2021
Merged

[pull] master from angular:master#165
pull[bot] merged 2 commits intoadmariner:masterfrom
angular:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Sep 27, 2021

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

clydin and others added 2 commits September 27, 2021 06:45
…ze` usage

With the Angular CLI currently being a CommonJS package, this change uses a dynamic import to load `@angular/localize` which may be ESM. CommonJS code can load ESM code via a dynamic import. Unfortunately, TypeScript will currently, unconditionally downlevel dynamic import into a require call. require calls cannot load ESM code and will result in a runtime error. To workaround this, a Function constructor is used to prevent TypeScript from changing the dynamic import. Once TypeScript provides support for keeping the dynamic import this workaround can be dropped and replaced with a standard dynamic import.
@pull pull bot added the ⤵️ pull label Sep 27, 2021
@pull pull bot merged commit d2e302c into admariner:master Sep 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants