Skip to content

Remove RenameDeprecatedMethodCallRector - #8485

Merged
TomasVotruba merged 1 commit into
mainfrom
remove-rename-deprecated-method-call-rector
Sep 11, 2026
Merged

Remove RenameDeprecatedMethodCallRector#8485
TomasVotruba merged 1 commit into
mainfrom
remove-rename-deprecated-method-call-rector

Conversation

@TomasVotruba

Copy link
Copy Markdown
Member

Removes RenameDeprecatedMethodCallRector along with its DeprecatedMethodCallReplacementResolver helper and tests.

Inferring a replacement method from free-text @deprecated descriptions is too risky - the heuristic can rename onto the wrong method. Explicit per-version rename rule registration (RenameMethodRector) is safer and predictable.

The rule was not part of any set, so no set changes are needed.

https://claude.ai/code/session_01JJoiy9Q76AadRMLSz6Xfj3

Too risky to infer replacements from @deprecated descriptions; explicit per-version rename rule registration is safer and predictable.

Claude-Session: https://claude.ai/code/session_01JJoiy9Q76AadRMLSz6Xfj3
@TomasVotruba
TomasVotruba merged commit c6d15cd into main Sep 11, 2026
43 of 44 checks passed
@TomasVotruba
TomasVotruba deleted the remove-rename-deprecated-method-call-rector branch September 11, 2026 18:55
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.

1 participant