Skip to content
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

Improve rendering of Markdown details #393

Merged
merged 2 commits into from
Nov 11, 2024
Merged

Improve rendering of Markdown details #393

merged 2 commits into from
Nov 11, 2024

Conversation

uhafner
Copy link
Owner

@uhafner uhafner commented Nov 10, 2024

Add support for all new PMD metrics (see pmd/pmd@main...uhafner:pmd:7.7.0-patch-2). These new PMD metrics have been introduced in the bachelor theses of Maximilian Waidelich here at the University of Applied Sciences Munich, see #119.

The following metrics are now supported:

📐   Toplevel Metrics

Icon Name Total Min Max Mean Median
🌀 Cyclomatic Complexity 355 1 8 1.73 1
🧠 Cognitive Complexity 172 0 11 0.84 0
📏 Lines of Code 3859 1 35 6.52 1
📝 Non Commenting Source Statements 1199 1 21 3.81 1
🔭 Access to foreign data 87 0 6 0.32 0
🔗 Class cohesion 0 0.00% 71.43% 13.59% 0.00%
📤 Fan out 224 0 13 1.78 0
📲 Number of accessors 14 0 2 0.54 0
🏋️ Weight of a class 1 0.00% 100.00% 83.65% 0.00%
📐 Weighted method count 354 3 46 14.75 3
N-Path Complexity 432 1 30 2.11 1

For each metric, minimum, maximum and mean is rendered now. Additionally, the icons for the coverage types and analysis tools are shown.

For each metric, minimum, maximum and mean is rendered now.
@uhafner uhafner added the enhancement Enhancement of existing functionality label Nov 10, 2024
Copy link

codecov bot commented Nov 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.96%. Comparing base (bfad2fb) to head (2f76c27).
Report is 4 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #393      +/-   ##
============================================
+ Coverage     85.81%   85.96%   +0.15%     
- Complexity      479      489      +10     
============================================
  Files            30       30              
  Lines          1403     1418      +15     
  Branches         82       82              
============================================
+ Hits           1204     1219      +15     
  Misses          167      167              
  Partials         32       32              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

☀️   Quality Monitor

   🚦   Tests: 98% successful (❌ 4 failed, ✔️ 163 passed)
   〰️   Line Coverage: 88% (167 missed lines)
   ➰   Branch Coverage: 85% (58 missed branches)
   PIT   Mutation Coverage: 0% (0 survived mutations)
   💪   Test Strength: 0% (0 survived mutations in tested code)
   CheckStyle   CheckStyle: No warnings
   PMD   PMD: 2 warnings (normal: 2)
   SpotBugs   SpotBugs: No bugs
   🐛   Error Prone: 2 bugs (normal: 2)
   OWASP Dependency Check   OWASP Dependency Check: No vulnerabilities
   🌀   Cyclomatic Complexity: 576
   🧠   Cognitive Complexity: 226
   📝   Non Commenting Source Statements: 2008
   ➿   N-Path Complexity: 1049


🚧   Error Messages

Quality Monitor Errors:
No matching report files found when using pattern '**/target/pit-reports/mutations.xml'! Configuration error for 'Mutation Coverage'?
No matching report files found when using pattern '**/target/pit-reports/mutations.xml'! Configuration error for 'Test Strength'?
No matching report files found when using pattern '**/target/dependency-check-report.json'! Configuration error for 'OWASP Dependency Check'?

Created by Quality Monitor v1.13.0 (#d2d7e89). More details are shown in the GitHub Checks Result.

Copy link

☀️   Quality Monitor

   🚦   Tests: 100% successful (✔️ 167 passed)
   〰️   Line Coverage: 88% (167 missed lines)
   ➰   Branch Coverage: 85% (58 missed branches)
   PIT   Mutation Coverage: 79% (168 survived mutations)
   💪   Test Strength: 88% (84 survived mutations in tested code)
   CheckStyle   CheckStyle: No warnings
   PMD   PMD: 2 warnings (normal: 2)
   SpotBugs   SpotBugs: No bugs
   🐛   Error Prone: 2 bugs (normal: 2)
   OWASP Dependency Check   OWASP Dependency Check: 1 vulnerability (normal: 1)
   🌀   Cyclomatic Complexity: 577
   🧠   Cognitive Complexity: 227
   📝   Non Commenting Source Statements: 2010
   ➿   N-Path Complexity: 1050


Created by Quality Monitor v1.13.0 (#d2d7e89). More details are shown in the GitHub Checks Result.

Copy link

☀️   Quality Monitor

   🚦   Tests: 100% successful (✔️ 167 passed)
   〰️   Line Coverage: 88% (167 missed lines)
   ➰   Branch Coverage: 85% (58 missed branches)
   PIT   Mutation Coverage: 79% (168 survived mutations)
   💪   Test Strength: 88% (84 survived mutations in tested code)
   CheckStyle   CheckStyle: No warnings
   PMD   PMD: 1 warning (normal: 1)
   SpotBugs   SpotBugs: No bugs
   🐛   Error Prone: 2 bugs (normal: 2)
   OWASP Dependency Check   OWASP Dependency Check: 1 vulnerability (normal: 1)
   🌀   Cyclomatic Complexity: 566
   🧠   Cognitive Complexity: 227
   📝   Non Commenting Source Statements: 2010
   ➿   N-Path Complexity: 1050


Created by Quality Monitor v1.13.0 (#d2d7e89). More details are shown in the GitHub Checks Result.

Copy link

☀️   Quality Monitor

   🚦   Tests: 100% successful (✔️ 168 passed)
   〰️   Line Coverage: 88% (167 missed lines)
   ➰   Branch Coverage: 85% (58 missed branches)
   PIT   Mutation Coverage: 79% (168 survived mutations)
   💪   Test Strength: 88% (84 survived mutations in tested code)
   CheckStyle   CheckStyle: No warnings
   PMD   PMD: No warnings
   SpotBugs   SpotBugs: No bugs
   🐛   Error Prone: 3 bugs (normal: 3)
   OWASP Dependency Check   OWASP Dependency Check: 1 vulnerability (normal: 1)
   🌀   Cyclomatic Complexity: 566
   🧠   Cognitive Complexity: 227
   📝   Non Commenting Source Statements: 2010
   ➿   N-Path Complexity: 1050


Created by Quality Monitor v1.13.0 (#d2d7e89). More details are shown in the GitHub Checks Result.

@uhafner uhafner merged commit 651847c into main Nov 11, 2024
16 checks passed
@uhafner uhafner deleted the all-metrics branch November 11, 2024 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement of existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant