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
Feature/2708/show changed files in delta mode #2890
Conversation
[Less symmetry in expected result when counting added+deleted files]
[check english grammar for plural/singular edge cases]
[edge case of folder with 0 files is now displayed with correct grammar]
[Text shows correctly that a file changed if and only a metric changed]
…://github.com/MaibornWolff/codecharta into feature/2708/show-changed-files-in-delta-mode
[ .metricsChanged to .changed ]
[For-loop instead of lambda-functions for perfomance in delta generator]
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.
Is the draft mode still intentional? :-)
It already looks very good to me! I just left a few minor comments and the changelog entry should be added as well.
[Using set instead of array to improve performance]
[0 should equal undefined when comparing attributes]
Feel free to push the button and merge on final approve. : ) |
Would you please have a look on dataMock.ts. |
[Snapshots use https and mocks have correct propery order]
[Reduce duplicated lines marked by sonarqube]
[Extract fileMeta proeprty to reduce code duplication]
[Extracted default file and default fileMap]
[Reduce code duplication for sonar qube]
[CodeCharta Analysis] Kudos, SonarCloud Quality Gate passed! |
[CodeCharta Visualization] Kudos, SonarCloud Quality Gate passed! |
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.
LGTM 🚀
Display number of files with metric changes in delta mode
Issue: #2708
Description
Descriptive pull request text, answering:
Screenshots or gifs