-
-
Notifications
You must be signed in to change notification settings - Fork 344
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
New LongMethodName Rule #486
Comments
I find this idea very useful. It would make me happy if you could implement it. Thank you. |
Everybody is welcome to create a PR that implements this rule 😃 Please say it if you want to implement id and also if you need some help. |
I finished implementation. But I suspect users might also need some pattern/regexp exception such as |
@kylekatarnls |
So I got carried away a bit again and made a pattern system for exceptions property. As I'm already a bit frustrated by the inconsistency of how we handle exceptions across rules, so I made a generic utility for it to be more easily harmonized in the next major version. #1045 |
I think we should ignore |
At the moment, we have:
https://phpmd.org/rules/index.html#naming-rules
But we do not have a
LongMethodName
rule.We should consider adding one, as an extensively long method name could be a sign of doing too much at once.
The text was updated successfully, but these errors were encountered: