Skip to content

Upgrade phpstan and psalm#4419

Closed
simPod wants to merge 1 commit intodoctrine:masterfrom
simPod:phpstan
Closed

Upgrade phpstan and psalm#4419
simPod wants to merge 1 commit intodoctrine:masterfrom
simPod:phpstan

Conversation

@simPod
Copy link
Copy Markdown
Contributor

@simPod simPod commented Nov 11, 2020

#4403 for master

message: '~^Parameter #1 \$expected of static method PHPUnit\\Framework\\Assert::assertInstanceOf\(\) expects class-string<Doctrine\\DBAL\\Connection&PHPUnit\\Framework\\MockObject\\MockObject>, class-string<Doctrine\\DBAL\\Connection>&class-string<PHPUnit\\Framework\\MockObject\\MockObject> given\.$~'
path: %currentWorkingDirectory%/tests/DriverManagerTest.php

-
Copy link
Copy Markdown
Member

@morozov morozov Nov 12, 2020

Choose a reason for hiding this comment

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

I'd file this an issue with PHPStan (see phpstan/phpstan-mockery#14 for the reference). iterator_to_array() obviously may produce side effects.

@morozov morozov closed this Nov 26, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 25, 2022
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