-
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
Fix/3456/update mcc naming #3606
base: main
Are you sure you want to change the base?
Conversation
We need to investigate how this change will impact our online demo. |
a292da5
to
bf35d19
Compare
I rebased the branch to include the main changes |
I did some testing and here are some points that I found. We should evaluate if those changes are acceptable:
|
I think all these points should be addressed before merging this issue. i further found that we include other metrics that talk about mcc (the sourceMonitor has max_function_mcc and average_function_mcc) that should be changed as well here. |
bf35d19
to
a5a90c4
Compare
Quality Gate passed for 'CodeCharta Visualization'Issues Measures |
Quality Gate passed for 'CodeCharta Analysis'Issues Measures |
…criptors for sourceCodeParser #3606
…criptors for sourceCodeParser #3606
a3411d2
to
5ec6f26
Compare
Update MCC naming
Closes: #3456
Description
Temporarily adds 'formerly mcc' to the no longer renamed sonar complexity
Definition of Done
A PR is only ready for merge once all the following acceptance criteria are fulfilled:
Screenshots or gifs