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

Add phpstan-phpunit to composer.json #733

Merged
merged 1 commit into from
Jun 13, 2018
Merged

Add phpstan-phpunit to composer.json #733

merged 1 commit into from
Jun 13, 2018

Conversation

jwage
Copy link
Member

@jwage jwage commented Jun 12, 2018

Q A
Type improvement
BC Break no

Summary

Add phpstan/phpstan-phpunit to composer.json

@jwage jwage added this to the 2.0.0-alpha2 milestone Jun 12, 2018
@jwage jwage self-assigned this Jun 12, 2018
@localheinz
Copy link
Contributor

Does this also need a modification of phpstan.neon?

For reference, see https://github.com/phpstan/phpstan-phpunit#usage:

includes:
	- vendor/phpstan/phpstan-phpunit/extension.neon

@Majkl578
Copy link
Contributor

@localheinz Yes.

Copy link
Contributor

@Majkl578 Majkl578 left a comment

Choose a reason for hiding this comment

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

phpstan.neon needs to be updated (see discussion)

@jwage jwage force-pushed the phpstan-phpunit branch from cbab7d8 to 94167ec Compare June 13, 2018 14:31
@jwage
Copy link
Member Author

jwage commented Jun 13, 2018

Whoops. Forgot that. It is updated now.

Copy link
Contributor

@localheinz localheinz left a comment

Choose a reason for hiding this comment

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

👍

phpstan.neon Outdated
@@ -5,3 +5,4 @@ parameters:

includes:
- vendor/phpstan/phpstan-strict-rules/rules.neon
- vendor/phpstan/phpstan-phpunit/extension.neon
Copy link
Contributor

Choose a reason for hiding this comment

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

Probably not that important, but in case more rules will be imported, I recommend to keep includes sorted, perhaps using the Lines Sorter plugin for PhpStorm.

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixing. Good call.

@localheinz
Copy link
Contributor

Build failure appears to be unrelated; restarting the build might fix it.

@jwage jwage force-pushed the phpstan-phpunit branch from 94167ec to 60e41d4 Compare June 13, 2018 15:09
@jwage jwage merged commit 732e0fe into master Jun 13, 2018
@jwage jwage deleted the phpstan-phpunit branch June 13, 2018 15:37
@jwage jwage modified the milestones: 2.0.0-beta1, 2.0.0 Aug 20, 2018
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.

4 participants