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

Introduce PHPStan #83

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

szepeviktor
Copy link
Contributor

Fixes #52

@szepeviktor
Copy link
Contributor Author

szepeviktor commented Jan 25, 2020

I think some code needs to updated to recognize array types and array shapes in PHPDoc.
https://travis-ci.com/webimpress/coding-standard/jobs/280021006

@michalbundyra
Copy link
Member

@szepeviktor Thanks for the PR. This is the problem that current rules does not support generics and intersection types (yet). Because of that it might be difficult to apply proper static analysis tools. Maybe vimeo/psalm will be easier as there we can define psalm types on properties etc... Haven't tried it, tho.

@szepeviktor
Copy link
Contributor Author

@michalbundyra We could go @phpstan-param. Should we?

@michalbundyra
Copy link
Member

@szepeviktor

We could go @phpstan-param. Should we?

Yeah... I think it is the best solution, at least for now.

@szepeviktor
Copy link
Contributor Author

szepeviktor commented Jan 25, 2020

Done. 🍏

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.

Benefit from static analysis
2 participants