Skip to content

fix(core): throw if standalone components are present in @NgModule.bootstrap#45825

Closed
AndrewKushnir wants to merge 1 commit intoangular:mainfrom
AndrewKushnir:standalone_bootstrap_ngmodule
Closed

fix(core): throw if standalone components are present in @NgModule.bootstrap#45825
AndrewKushnir wants to merge 1 commit intoangular:mainfrom
AndrewKushnir:standalone_bootstrap_ngmodule

Conversation

@AndrewKushnir
Copy link
Contributor

This commit updates the logic to detect a situation when a standalone component is used in the NgModule-based bootstrap (@NgModule.bootstrap). Both AOT and JIT compilers are updated to handle this situation.

PR Type

What kind of change does this PR introduce?

  • Bugfix

Does this PR introduce a breaking change?

  • Yes
  • No

@AndrewKushnir AndrewKushnir added state: WIP area: core Issues related to the framework runtime target: major This PR is targeted for the next major release cross-cutting: standalone Issues related to the NgModule-less world labels Apr 30, 2022
@AndrewKushnir AndrewKushnir added this to the v14-candidates milestone Apr 30, 2022
@AndrewKushnir AndrewKushnir force-pushed the standalone_bootstrap_ngmodule branch 3 times, most recently from 28bba7a to 39db1f2 Compare April 30, 2022 05:24
@AndrewKushnir AndrewKushnir added action: review The PR is still awaiting reviews from at least one requested reviewer and removed state: WIP labels Apr 30, 2022
@AndrewKushnir AndrewKushnir marked this pull request as ready for review April 30, 2022 05:25
@pullapprove pullapprove bot requested review from alxhub, dylhunn and jessicajaniuk and removed request for pkozlowski-opensource April 30, 2022 05:25
@AndrewKushnir AndrewKushnir force-pushed the standalone_bootstrap_ngmodule branch from 39db1f2 to b5358a4 Compare April 30, 2022 05:48
@pullapprove pullapprove bot requested a review from atscott April 30, 2022 05:48
Copy link
Member

@pkozlowski-opensource pkozlowski-opensource left a comment

Choose a reason for hiding this comment

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

LGTM

@AndrewKushnir AndrewKushnir added action: presubmit The PR is in need of a google3 presubmit and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Apr 30, 2022
@AndrewKushnir
Copy link
Contributor Author

Presubmit.

@AndrewKushnir AndrewKushnir force-pushed the standalone_bootstrap_ngmodule branch from b5358a4 to 6ae126a Compare May 2, 2022 15:44
dylhunn
dylhunn previously approved these changes May 2, 2022
Copy link
Contributor

@dylhunn dylhunn left a comment

Choose a reason for hiding this comment

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

reviewed-for: public-api

…ootstrap`

This commit updates the logic to detect a situation when a standalone component is used in the NgModule-based bootstrap (`@NgModule.bootstrap`). Both AOT and JIT compilers are updated to handle this situation.
@AndrewKushnir AndrewKushnir force-pushed the standalone_bootstrap_ngmodule branch from 6ae126a to 93d94bd Compare May 2, 2022 16:50
Copy link
Contributor

@atscott atscott left a comment

Choose a reason for hiding this comment

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

reviewed-for: public-api

@AndrewKushnir AndrewKushnir removed the action: presubmit The PR is in need of a google3 presubmit label May 2, 2022
@AndrewKushnir AndrewKushnir added action: merge The PR is ready for merge by the caretaker merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note labels May 2, 2022
@AndrewKushnir
Copy link
Contributor Author

Merge-assistance: the scope of the public-api approval is only a new error code, so 2 approvals should be sufficient.

@dylhunn dylhunn dismissed stale reviews from pkozlowski-opensource and themself via 93d94bd May 2, 2022 18:42
@dylhunn
Copy link
Contributor

dylhunn commented May 2, 2022

This PR was merged into the repository by commit fde4942.

@dylhunn dylhunn closed this in fde4942 May 2, 2022
@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 Jun 2, 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 area: core Issues related to the framework runtime cross-cutting: standalone Issues related to the NgModule-less world merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants