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

Fix phpstan false positives #3200

Merged
merged 1 commit into from
Oct 22, 2019
Merged

Fix phpstan false positives #3200

merged 1 commit into from
Oct 22, 2019

Conversation

soyuka
Copy link
Member

@soyuka soyuka commented Oct 22, 2019

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tickets na
License MIT
Doc PR na

False positives are reported on the path of the file

@soyuka soyuka force-pushed the fix-phpstan branch 3 times, most recently from a657312 to ac44493 Compare October 22, 2019 11:36
phpstan.neon.dist Outdated Show resolved Hide resolved
@teohhanhui
Copy link
Contributor

teohhanhui commented Oct 22, 2019

Actually, let's just exclude this new version of phpstan for now?

EDIT: Opened phpstan/phpstan#2532

phpstan.neon.dist Outdated Show resolved Hide resolved
@soyuka soyuka merged commit 9425f08 into api-platform:2.5 Oct 22, 2019
@soyuka soyuka deleted the fix-phpstan branch October 22, 2019 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants