Skip to content

Conversation

@crisbeto
Copy link
Member

Adjusts the return type of FormBuilder.group to work around microsoft/TypeScript#60506.

Adjusts the return type of `FormBuilder.group` to work around microsoft/TypeScript#60506.
@crisbeto crisbeto added target: patch This PR is targeted for the next patch release action: global presubmit The PR is in need of a google3 global presubmit labels Nov 19, 2024
@pullapprove pullapprove bot requested a review from alxhub November 19, 2024 09:08
@ngbot ngbot bot added this to the Backlog milestone Nov 19, 2024
@crisbeto crisbeto requested review from AndrewKushnir and removed request for alxhub November 19, 2024 09:13
Copy link
Member

@alxhub alxhub 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
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: public-api

@crisbeto
Copy link
Member Author

Passing TGP after one manual tweak. Will unblock this PR once the team accepts the fix.

@crisbeto crisbeto added action: merge The PR is ready for merge by the caretaker and removed action: global presubmit The PR is in need of a google3 global presubmit labels Nov 19, 2024
@AndrewKushnir
Copy link
Contributor

This PR was merged into the repository by commit 4a18dc0.

The changes were merged into the following branches: main, 19.0.x

AndrewKushnir pushed a commit that referenced this pull request Nov 19, 2024
Adjusts the return type of `FormBuilder.group` to work around microsoft/TypeScript#60506.

PR Close #58731
crisbeto added a commit to crisbeto/angular that referenced this pull request Nov 20, 2024
Reworks the changes from angular#58731, because they didn't cover all use cases.
thePunderWoman pushed a commit that referenced this pull request Nov 21, 2024
Reworks the changes from #58731, because they didn't cover all use cases.

PR Close #58782
thePunderWoman pushed a commit that referenced this pull request Nov 21, 2024
Reworks the changes from #58731, because they didn't cover all use cases.

PR Close #58782
@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 Dec 20, 2024
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: forms target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants