-
-
Notifications
You must be signed in to change notification settings - Fork 432
Fix typos #7120
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
Fix typos #7120
Conversation
| declare(strict_types=1); | ||
|
|
||
| namespace Rector\Tests\Issues\IssueDoubleNestedAnnotatoinDocBlock; | ||
| namespace Rector\Tests\Issues\IssueDoubleNestedAnnotationDocBlock; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I doubt that changing the class name and namespace of a test class is considered a BC break?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems fine
| */ | ||
| private function resolveName(string $prefix, string $tag, UseItem $useItem): string | ||
| { | ||
| // useuse can be renamed on the fly, so just in case, use the original one |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just change useuse to UseItem
| private function isUseMatchingName(string $tag, UseItem $useItem): bool | ||
| { | ||
| // useuse can be renamed on the fly, so just in case, use the original one | ||
| // use can be renamed on the fly, so just in case, use the original one |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
UseItem
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
Thank you @mttsch |
|
This pull request has been automatically locked because it has been closed for 150 days. Please open a new PR if you want to continue the work. |
No description provided.