Skip to content

[PostRector] Allow unused alias removal from Use_ on UnusedImportRemovingPostRector#3477

Merged
samsonasik merged 12 commits intomainfrom
add-alias-support-remove-unused-use
Mar 13, 2023
Merged

[PostRector] Allow unused alias removal from Use_ on UnusedImportRemovingPostRector#3477
samsonasik merged 12 commits intomainfrom
add-alias-support-remove-unused-use

Conversation

@samsonasik
Copy link
Copy Markdown
Member

@samsonasik samsonasik commented Mar 12, 2023

For usage of :

$rectorConfig->removeUnusedImports();

on aliased use statements.

@samsonasik
Copy link
Copy Markdown
Member Author

All checks have passed 🎉 @TomasVotruba it is ready for review.

@samsonasik
Copy link
Copy Markdown
Member Author

@TomasVotruba I am merging it ;)

@samsonasik samsonasik merged commit c240c69 into main Mar 13, 2023
@samsonasik samsonasik deleted the add-alias-support-remove-unused-use branch March 13, 2023 22:27
@TomasVotruba
Copy link
Copy Markdown
Member

Feel free to merge these use/alias/imports PRs as you go 👍 :)

samsonasik added a commit that referenced this pull request May 8, 2023
…vingPostRector (#3477)

* [PostRector] Allow unused alias removal from Use_ on UnusedImportRemovingPostRector

* uniquate name

* check prefixed

* clean up

* clean up

* clean up

* Fix phpstan

* fix

* aliased part

* aliased part

* Final touch: array_unique() in final merge

* final touch: add fixture aliased part on docblock
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants