Skip to content

[AutoImport] Avoid used import into param that skipped from removal rule#8078

Merged
TomasVotruba merged 6 commits into
mainfrom
avoid-removed
Jun 23, 2026
Merged

[AutoImport] Avoid used import into param that skipped from removal rule#8078
TomasVotruba merged 6 commits into
mainfrom
avoid-removed

Conversation

@samsonasik

@samsonasik samsonasik commented Jun 22, 2026

Copy link
Copy Markdown
Member

This happen when:

  • There is enabled RemoveUnusedConstructorParamRector
  • Skipped on the target fixture
  • Apply auto import with removal unused use (marked as removable, which should not)

Ref #8043 (comment)

@samsonasik

Copy link
Copy Markdown
Member Author

@TomasVotruba it seems due to refactor() calls for unused skip config feature on 2.5.x

@samsonasik

Copy link
Copy Markdown
Member Author

Fixed /cc @ruudk @TomasVotruba

@samsonasik samsonasik requested a review from TomasVotruba June 22, 2026 14:54
@samsonasik

Copy link
Copy Markdown
Member Author

@TomasVotruba ready 👍

@TomasVotruba TomasVotruba merged commit a7b3eb3 into main Jun 23, 2026
67 checks passed
@TomasVotruba TomasVotruba deleted the avoid-removed branch June 23, 2026 10:23
@TomasVotruba

Copy link
Copy Markdown
Member

Thank you

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.

5 participants