Skip to content

fix(docs-infra): include tslib into SystemJS config in upgrade-module example app#45706

Closed
AndrewKushnir wants to merge 1 commit intoangular:masterfrom
AndrewKushnir:systemjs-config
Closed

fix(docs-infra): include tslib into SystemJS config in upgrade-module example app#45706
AndrewKushnir wants to merge 1 commit intoangular:masterfrom
AndrewKushnir:systemjs-config

Conversation

@AndrewKushnir
Copy link
Contributor

This commit updates the SystemJS for one of the example apps (the upgrade-module one) to include a resolution rule for the tslib. This is needed in case tslib is referenced from the framework code (for example in case of downleveling of some operators). This makes it consistent with other example app configs.

This issue was causing Protractor errors in the PR #45674 when native async/await were used in the code and downlevelled to use __awaiter helper from tslib.

PR Type

What kind of change does this PR introduce?

  • angular.io application / infrastructure changes

Does this PR introduce a breaking change?

  • Yes
  • No

…ule` example app

This commit updates the SystemJS for one of the example apps (the `upgrade-module` one) to include a resolution rule for the `tslib`. This is needed in case `tslib` is referenced from the framework code (for example in case of downleveling of some operators). This makes it consistent with other example app configs.
@AndrewKushnir AndrewKushnir added action: review The PR is still awaiting reviews from at least one requested reviewer comp: docs-infra target: patch This PR is targeted for the next patch release labels Apr 21, 2022
@AndrewKushnir AndrewKushnir requested a review from gkalpak April 21, 2022 06:10
@ngbot ngbot bot modified the milestone: Backlog Apr 21, 2022
Copy link
Member

@gkalpak gkalpak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch 👍

@gkalpak gkalpak added type: bug/fix action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Apr 21, 2022
@atscott
Copy link
Contributor

atscott commented Apr 21, 2022

This PR was merged into the repository by commit 9292953.

@atscott atscott closed this in 9292953 Apr 21, 2022
atscott pushed a commit that referenced this pull request Apr 21, 2022
…ule` example app (#45706)

This commit updates the SystemJS for one of the example apps (the `upgrade-module` one) to include a resolution rule for the `tslib`. This is needed in case `tslib` is referenced from the framework code (for example in case of downleveling of some operators). This makes it consistent with other example app configs.

PR Close #45706
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators May 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release type: bug/fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants