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
Implementation of attributeDescriptors, a new json field that describes used metrics #3091
Conversation
analysis/model/src/main/kotlin/de/maibornwolff/codecharta/model/AttributeDescriptor.kt
Outdated
Show resolved
Hide resolved
…o files are left behind
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general it is LGTM! I did not have a look at the Unit Tests so far, but I am planning to do this (it should not be a blocker for the merge though).
As the next step I would prioritize to display the new descriptors in the Frontend. |
...EdgeFilter/src/test/kotlin/de/maibornwolff/codecharta/filter/edgefilter/ProjectMergerTest.kt
Show resolved
Hide resolved
...ifier/src/test/kotlin/de/maibornwolff/codecharta/filter/structuremodifier/NodeRemoverTest.kt
Outdated
Show resolved
Hide resolved
...c/test/kotlin/de/maibornwolff/codecharta/filter/structuremodifier/SubProjectExtractorTest.kt
Outdated
Show resolved
Hide resolved
...mport/CSVImporter/src/test/kotlin/de/maibornwolff/codecharta/importer/csv/CSVImporterTest.kt
Outdated
Show resolved
Hide resolved
...CSVImporter/src/test/kotlin/de/maibornwolff/codecharta/importer/csv/CSVProjectBuilderTest.kt
Outdated
Show resolved
Hide resolved
...gParser/src/test/kotlin/de/maibornwolff/codecharta/importer/gitlogparser/GitLogParserTest.kt
Show resolved
Hide resolved
[CodeCharta Analysis] Kudos, SonarCloud Quality Gate passed! |
[CodeCharta Visualization] Kudos, SonarCloud Quality Gate passed! |
attributeDescriptors basic implementation #1066
Issue: #1066
Description
This adds a new field to the produced json files from cc-analysis. This field functions as a map to describe the various attributes produced from the different importers/parsers. Those attribute/metric explanations can then be displayed in the cc-visualization to give the user additional information.
Screenshots or gifs
This is still a draft and only includes the outlines of the implementation, some features are not done or incomplete.
If there are any recommendations for changes, let me know 👍