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

#69 re-introduced PHPStan static analysis #95

Merged
merged 26 commits into from
Aug 21, 2018

Conversation

Ocramius
Copy link
Member

@Ocramius Ocramius commented Aug 21, 2018

Fixes #69
Provides static analysis checks also for test/

@Ocramius Ocramius added this to the 1.1.0 milestone Aug 21, 2018
@Ocramius Ocramius self-assigned this Aug 21, 2018
…omposer dependency discovery logic
…rts due to un-recognised type declaration
…rts due to un-recognised type declaration
…rts due to un-recognised type declaration
…rts due to un-recognised type declaration
… that led to `string|bool` type explosions
@Ocramius Ocramius force-pushed the feature/#96-phpstan-static-analysis branch from 4cf0116 to 4a05e71 Compare August 21, 2018 10:28
@Ocramius Ocramius changed the title #96 re-introduced PHPStan static analysis #69 re-introduced PHPStan static analysis Aug 21, 2018
@Ocramius Ocramius merged commit 34f627b into master Aug 21, 2018
@Ocramius Ocramius deleted the feature/#96-phpstan-static-analysis branch August 21, 2018 10:57
@@ -0,0 +1,8 @@
parameters:
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not phpstan.neon.dist?

Copy link
Member Author

Choose a reason for hiding this comment

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

Missed it, making a new PR

Copy link
Member Author

Choose a reason for hiding this comment

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

#99

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants