-
-
Notifications
You must be signed in to change notification settings - Fork 157
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
Integrate sidz/phpstan-rules
to avoid magic numbers in our code base
#1861
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please add tests to make sure this change works as expected?
unrelated, but thanks for the service |
sidz/phpstan-rules
to avoid magic numbers in our code basesidz/phpstan-rules
to avoid magic numbers in our code base
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Thank you, @sidz |
I propose to add a 3rd-party package to documentation about detecting Magic Numbers in the code: https://github.com/sidz/phpstan-rules This is a replacement for https://github.com/povils/phpmnd (standalone CLI tool) We have already integrated in several projects at our company and I personally add it to @infection (infection/infection#1861) Would love to share this great package with more developers.
I propose to add a 3rd-party package to documentation about detecting Magic Numbers in the code: https://github.com/sidz/phpstan-rules This is a replacement for https://github.com/povils/phpmnd (standalone CLI tool) We have already integrated in several projects at our company and I personally add it to @infection (infection/infection#1861) Would love to share this great package with more developers.
This is a PHPStan plugin from our @sidz
No magic numbers!