Skip to content

refactor(common): improve typing of ngComponentOutletContent#63674

Closed
JeanMeche wants to merge 1 commit intoangular:mainfrom
JeanMeche:ng-component-outlet-content-type
Closed

refactor(common): improve typing of ngComponentOutletContent#63674
JeanMeche wants to merge 1 commit intoangular:mainfrom
JeanMeche:ng-component-outlet-content-type

Conversation

@JeanMeche
Copy link
Copy Markdown
Member

@JeanMeche JeanMeche commented Sep 9, 2025

BREAKING CHANGE ngComponentOutletContent is now of type Node[][] | undefined instead of any[][]

fixes #63538

@angular-robot angular-robot bot added the area: common Issues related to APIs in the @angular/common package label Sep 9, 2025
@ngbot ngbot bot added this to the Backlog milestone Sep 9, 2025
Dropping `any` in favor of `Node` for better type safety and clarity.

BREAKING CHANGE: `ngComponentOutletContent` is now of type `Node[][] | undefined` instead of `any[][] | undefined`.

fixes angular#63538
@JeanMeche JeanMeche force-pushed the ng-component-outlet-content-type branch from 5a6416d to a1c0727 Compare September 10, 2025 14:02
@angular-robot angular-robot bot added the detected: breaking change PR contains a commit with a breaking change label Sep 10, 2025
@JeanMeche JeanMeche added breaking changes and removed detected: breaking change PR contains a commit with a breaking change labels Sep 10, 2025
@JeanMeche
Copy link
Copy Markdown
Member Author

TGP is "green"

@JeanMeche JeanMeche marked this pull request as ready for review September 10, 2025 15:04
@JeanMeche JeanMeche added the target: major This PR is targeted for the next major release label Sep 10, 2025
Copy link
Copy Markdown
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

Copy link
Copy Markdown
Contributor

@thePunderWoman thePunderWoman 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: fw-general, public-api

@JeanMeche JeanMeche added the action: merge The PR is ready for merge by the caretaker label Sep 10, 2025
@thePunderWoman thePunderWoman added detected: breaking change PR contains a commit with a breaking change and removed detected: breaking change PR contains a commit with a breaking change labels Sep 10, 2025
@thePunderWoman
Copy link
Copy Markdown
Contributor

@JeanMeche I think this needs a commit message tweak so that the detected: breaking change label works. I think we need that for release notes.

@JeanMeche JeanMeche added the detected: breaking change PR contains a commit with a breaking change label Sep 10, 2025
@thePunderWoman
Copy link
Copy Markdown
Contributor

This PR was merged into the repository. The changes were merged into the following branches:

@angular-automatic-lock-bot
Copy link
Copy Markdown

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 Oct 11, 2025
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: common Issues related to APIs in the @angular/common package breaking changes detected: breaking change PR contains a commit with a breaking change target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Typing of NgComponentOutlet input ngComponentOutletContent is incorrect

3 participants