Skip to content

Conversation

@mttsch
Copy link
Contributor

@mttsch mttsch commented Aug 8, 2025

No description provided.

declare(strict_types=1);

namespace Rector\Tests\Issues\IssueDoubleNestedAnnotatoinDocBlock;
namespace Rector\Tests\Issues\IssueDoubleNestedAnnotationDocBlock;
Copy link
Contributor Author

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?

Copy link
Member

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
Copy link
Member

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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

UseItem

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@samsonasik samsonasik merged commit 66957ed into rectorphp:main Aug 8, 2025
47 checks passed
@samsonasik
Copy link
Member

Thank you @mttsch

@mttsch mttsch deleted the bugfix/typos branch August 8, 2025 12:25
@github-actions
Copy link
Contributor

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 11, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants