You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Both the warning and the error message displayed by the Generic.Metrics.CyclomaticComplexity sniff could be improved if they included the name of the function that caused the error.
Describe the solution you'd like
Currently, the sniff displays the following error messages:
72 | WARNING | Function's cyclomatic complexity (20) exceeds 10; consider refactoring the function
118 | ERROR | Function's cyclomatic complexity (21) exceeds allowed maximum of 20
I'm suggesting we update them to something like (note that I also added "the" before "allowed maximum"):
72 | WARNING | Function complexityTwenty()'s cyclomatic complexity (20) exceeds 10; consider refactoring the function
118 | ERROR | Function complexityTwentyOne()'s cyclomatic complexity (21) exceeds allowed maximum of 20
Additional context (optional)
Below there are links to the places where the two messages are defined:
Is your feature request related to a problem?
Both the warning and the error message displayed by the
Generic.Metrics.CyclomaticComplexity
sniff could be improved if they included the name of the function that caused the error.Describe the solution you'd like
Currently, the sniff displays the following error messages:
I'm suggesting we update them to something like (note that I also added "the" before "allowed maximum"):
Additional context (optional)
Below there are links to the places where the two messages are defined:
PHP_CodeSniffer/src/Standards/Generic/Sniffs/Metrics/CyclomaticComplexitySniff.php
Line 99 in da0f04b
PHP_CodeSniffer/src/Standards/Generic/Sniffs/Metrics/CyclomaticComplexitySniff.php
Line 106 in da0f04b
The text was updated successfully, but these errors were encountered: