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
Update doctrine/coding-standard from 6.0.0 to 7.0.2 #514
Conversation
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.
Overall very good: needs some adjustments due to previously imprecise /** @var */
annotations around mock objects
$sourceLocator = $this | ||
->getMockBuilder(StringSourceLocator::class) | ||
->disableOriginalConstructor() | ||
->setMethods(['locateIdentifier']) | ||
->getMock(); | ||
assert($sourceLocator instanceof StringSourceLocator || $sourceLocator instanceof MockObject); |
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.
Should be &&
for these transformations
$mockReflector = $this->createMock(Reflector::class); | ||
assert($mockReflector instanceof Reflector || $mockReflector instanceof MockObject); |
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.
Since newer phpunit versions, we can drop these, since direct $this->createMock()
calls (without builder) can be correctly inferred AFAIK
Build https://travis-ci.org/Roave/BetterReflection/jobs/635186981 failed due to too old phpunit version: we can bump the minimum supported version safely |
🤞 |
LULZ CS is good though, so I would merge this as-is, and bring the new problem with me as a separate issue. |
Closes #506.
Closes #507.
Closes #510.