Skip to content
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

Added .phpstorm.meta.php for PHPUnit mocks #3607

Closed
wants to merge 1 commit into from

Conversation

morozov
Copy link
Member

@morozov morozov commented Jun 15, 2019

Q A
Type improvement
BC Break no

With this file in the root of the project, PhpStorm will be able to correctly determine the type of mock objects, therefore resulting in fewer false positive type errors and better code completion.

Screenshot from 2019-06-14 12-05-34

Copy link
Member

@Ocramius Ocramius left a comment

Choose a reason for hiding this comment

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

This is already part of PHPUnit 8.2.0 as per sebastianbergmann/phpunit#3708

@morozov
Copy link
Member Author

morozov commented Jun 15, 2019

Cool. I may have forgotten to update my local dependencies since it only worked with the added file.

@morozov morozov closed this Jun 15, 2019
@morozov morozov deleted the phpstorm-meta branch June 15, 2019 01:35
@morozov morozov self-assigned this Oct 8, 2019
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants