-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[Bridge\PhpUnit] Cannot autoload class "\Symfony\Bridge\PhpUnit\SymfonyTestsListener" #26562
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
Conversation
@@ -18,3 +18,7 @@ class_alias('Symfony\Bridge\PhpUnit\Legacy\SymfonyTestsListenerForV6', 'Symfony\ | |||
} else { | |||
class_alias('Symfony\Bridge\PhpUnit\Legacy\SymfonyTestsListenerForV7', 'Symfony\Bridge\PhpUnit\SymfonyTestsListener'); | |||
} | |||
|
|||
if (false){ |
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.
missing space after closing bracket
Sorry, something went wrong.
All reactions
@@ -18,3 +18,7 @@ class_alias('Symfony\Bridge\PhpUnit\Legacy\SymfonyTestsListenerForV6', 'Symfony\ | |||
} else { | |||
class_alias('Symfony\Bridge\PhpUnit\Legacy\SymfonyTestsListenerForV7', 'Symfony\Bridge\PhpUnit\SymfonyTestsListener'); | |||
} | |||
|
|||
if (false){ | |||
class SymfonyTestsListener{} |
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.
please put {}
on separate lines
class SymfonyTestsListener
{
}
Sorry, something went wrong.
All reactions
|
All reactions
-
👍 1 reaction
Sorry, something went wrong.
Updated |
All reactions
Sorry, something went wrong.
The build failed with |
All reactions
Sorry, something went wrong.
Thank you @yakobe. |
All reactions
Sorry, something went wrong.
…pUnit\SymfonyTestsListener" (Jake Bishop) This PR was merged into the 3.4 branch. Discussion ---------- [Bridge\PhpUnit] Cannot autoload class "\Symfony\Bridge\PhpUnit\SymfonyTestsListener" | Q | A | ------------- | --- | Branch? | 3.4 <!-- see below --> | Bug fix? | yes | New feature? | no <!-- don't forget to update src/**/CHANGELOG.md files --> | BC breaks? | no <!-- see https://symfony.com/bc --> | Deprecations? | no <!-- don't forget to update UPGRADE-*.md files --> | Tests pass? | yes <!-- please add some, will be required by reviewers --> | Fixed tickets | #26430 | License | MIT | Doc PR | - <!-- required for new features --> Allows composer to discover the class for autoloading when using `composer install --classmap-authoritative` Commits ------- 6b6fdab [Bridge\PhpUnit] Fix #26430 Cannot autoload listeners
Thank you @nicolas-grekas ! Now i can use it :) |
All reactions
Sorry, something went wrong.
nicolas-grekas
derrabus
Successfully merging this pull request may close these issues.
Allows composer to discover the class for autoloading when using
composer install --classmap-authoritative