-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
GroupImportFixer - fix breaking code when fixing root classes #5598
Conversation
@Leprechaunz you don't want to add that test in integration tests, but in |
use Foo\Bar; | ||
use Foo\Baz; | ||
use ReflectionClass; | ||
use ReflectionMethod; |
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.
Let's add one more test when all imported classes have leading \
.
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.
Good point, thanks
2849fa9
to
486b598
Compare
Thanks for fixing this bug @Leprechaunz. |
Thanks @Leprechaunz 😍 |
Fixes #5574