Suggestion: Include complexity level in noExcessiveCognitiveComplexity LSP diagnostic
#4012
Closed
chrisgrieser
started this conversation in
Ideas
Replies: 2 comments 1 reply
-
|
I agree. PR are welcome :) |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
haven't touched any rust code before, but this one change seems straightforward enough, so I made a PR: #4028 |
Beta Was this translation helpful? Give feedback.
1 reply
-
|
Pr has been merged |
Beta Was this translation helpful? Give feedback.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Please refactor this function to reduce its complexity score from 17 to the max allowed complexity 15.Excessive complexity detected. (biome: lint/complexity/noExcessiveCognitiveComplexity)Basically, the LSP diagnostic is missing the allowed and current complexity level. However, when tackling such a complexity issue, it is often useful to know how much complexity has been reduced by a change you made etc. Thus, it would be very useful to include the complexity levels in the LSP diagnostic as well.
Beta Was this translation helpful? Give feedback.
All reactions