Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✨ New Universal.UseStatements.NoUselessAliases sniff #244

Merged
merged 1 commit into from
Jun 18, 2023

Conversation

jrfnl
Copy link
Member

@jrfnl jrfnl commented Jun 18, 2023

New sniff to detect useless aliases in import use statements.

Aliasing something to the same name as the original construct is considered useless (though allowed in PHP).
Note: as OO and function names in PHP are case-insensitive, aliasing to the same name, using a different case is also considered useless.

Includes fixer.
Includes unit tests.
Includes documentation.

Fixes #233

New sniff to detect useless aliases in import use statements.

Aliasing something to the same name as the original construct is considered useless (though allowed in PHP).
Note: as OO and function names in PHP are case-insensitive, aliasing to the same name, using a different case is also considered useless.

Includes unit tests.
Includes documentation.

Fixes 233
@jrfnl jrfnl added this to the 1.1.0 milestone Jun 18, 2023
@jrfnl jrfnl merged commit 1d70047 into develop Jun 18, 2023
37 checks passed
@jrfnl jrfnl deleted the feature/new-no-useless-aliases-sniff branch June 18, 2023 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sniff to detect useless alias names
1 participant