-
Notifications
You must be signed in to change notification settings - Fork 30
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
MCC name is incorrect #3456
Comments
Some notes to cc, mcc |
Two ideas:
|
The two mentioned metrics are the same, no? I am used to also count && and || operators for a complexity metric. |
I thought (wrongly), that sonar uses MCC, but it is a modified version as you say. |
Yes, I think that is fine. It is unfortune that the acronym is I will create an issue for renaming I think there are a lot of hard references to |
Update/conclusion on the current situation for this issue: Our mcc metric is the complexity metric from sonar but renamed. The first idea was to not rename it anymore which would result in us having a 'new' sonar_complexity metric but no mcc metric anymore. As mcc is one of the most used metrics, this change should not be done before informing/discussing it with the stakeholders. This issue will be blocked until we decide on how to continue with this problem |
Yes we can stop renaming it to mcc and use the default name instead. In that specific case: sonar_complexity |
As proposed by @Nereboss we stop to rename it and add "(formerly MCC)" to it. |
Bug
Expected Behavior
WHEN
Looking at the legend
THEN
The MCC is described with the correct words (likely McCabe Complexity or Cyclomatic Complexity)
Actual Behavior
The MCC is described as Maximum Cyclomatic Complexity
Specifications
The text was updated successfully, but these errors were encountered: