Skip to content

[DeadCode] Deprecate RemoveParentDelegatingConstructorRector - #8755

Merged
TomasVotruba merged 2 commits into
mainfrom
tv-deprecate-remove-parent-delegating-constructor
Oct 21, 2026
Merged

TomasVotruba merged 2 commits into
mainfrom
tv-deprecate-remove-parent-delegating-constructor

Conversation

@TomasVotruba

Copy link
Copy Markdown
Member

Removing a delegating constructor is risky and such constructors are nearly always kept on purpose, so this rule is deprecated.

  • rule gutted to throw via DeprecatedInterface
  • removed from DeadCodeLevel set
  • dropped rule tests
  • dropped the rule from the property-promotion issue test config (the promotion guard still holds)

TomasVotruba and others added 2 commits October 21, 2026 22:58
Removing a delegating constructor is risky and such constructors are nearly always kept on purpose.
@TomasVotruba
TomasVotruba merged commit 847713c into main Oct 21, 2026
49 checks passed
@TomasVotruba
TomasVotruba deleted the tv-deprecate-remove-parent-delegating-constructor branch October 21, 2026 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants