Skip to content

fix(multiple): remove empty constructors#33048

Merged
crisbeto merged 1 commit intoangular:mainfrom
crisbeto:remove-constructors
Apr 8, 2026
Merged

fix(multiple): remove empty constructors#33048
crisbeto merged 1 commit intoangular:mainfrom
crisbeto:remove-constructors

Conversation

@crisbeto
Copy link
Copy Markdown
Member

@crisbeto crisbeto commented Apr 8, 2026

When we made the transition to inject, we added some constructors for backwards compatibility. These changes remove them since they weren't used for anything.

BREAKING CHANGE:

  • A bunch of constructors that with rest arguments have been removed. If you were extending Material/CDK components, you may have to update your super calls accordingly.

@crisbeto crisbeto added the target: minor This PR is targeted for the next minor release label Apr 8, 2026
@pullapprove pullapprove bot requested review from adolgachev and andrewseguin April 8, 2026 07:40
@angular-robot angular-robot bot added the detected: breaking change PR contains a commit with a breaking change label Apr 8, 2026
When we made the transition to `inject`, we added some constructors for backwards compatibility. These changes remove them since they weren't used for anything.

BREAKING CHANGE:
* A bunch of constructors that with rest arguments have been removed. If you were extending Material/CDK components, you may have to update your `super` calls accordingly.
@crisbeto crisbeto force-pushed the remove-constructors branch from 18ce62f to 5110216 Compare April 8, 2026 07:44
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Apr 8, 2026
@crisbeto crisbeto merged commit ce4c2c0 into angular:main Apr 8, 2026
26 checks passed
@crisbeto
Copy link
Copy Markdown
Member Author

crisbeto commented Apr 8, 2026

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker detected: breaking change PR contains a commit with a breaking change target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants