Skip to content

Optimize how we get project statistics#9217

Merged
schalkms merged 2 commits into
mainfrom
optimize-metrics-statistics
Apr 4, 2026
Merged

Optimize how we get project statistics#9217
schalkms merged 2 commits into
mainfrom
optimize-metrics-statistics

Conversation

@BraisGabin
Copy link
Copy Markdown
Member

@BraisGabin BraisGabin commented Apr 1, 2026

We were using a DetektVisitor to get a KtFile. But we have that KtFile directly so we can remove the DetektVisitor directly.

@BraisGabin BraisGabin mentioned this pull request Apr 1, 2026
Draft
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 1, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.90%. Comparing base (ff33b7d) to head (0b861db).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files 
@@             Coverage Diff              @@
##               main    #9217      +/-   ##
============================================
- Coverage     84.93%   84.90%   -0.03%     
+ Complexity     4433     4428       -5     
============================================
  Files           571      570       -1     
  Lines         12307    12286      -21     
  Branches       2694     2694              
============================================
- Hits          10453    10432      -21     
  Misses          683      683              
  Partials       1171     1171

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

github-advanced-security[bot]
github-advanced-security AI found potential problems Apr 1, 2026
View reviewed changes
Comment thread detekt-metrics/src/main/kotlin/dev/detekt/metrics/processors/ClassCountProcessor.kt Fixed
Comment thread detekt-metrics/src/main/kotlin/dev/detekt/metrics/processors/ClassCountProcessor.kt Fixed
Comment thread detekt-metrics/src/main/kotlin/dev/detekt/metrics/processors/ClassCountProcessor.kt Fixed
Comment thread detekt-metrics/src/main/kotlin/dev/detekt/metrics/processors/FunctionCountProcessor.kt Fixed
Comment thread detekt-metrics/src/main/kotlin/dev/detekt/metrics/processors/FunctionCountProcessor.kt Fixed
Comment thread detekt-metrics/src/main/kotlin/dev/detekt/metrics/processors/FunctionCountProcessor.kt Fixed
Comment thread detekt-metrics/src/test/kotlin/dev/detekt/metrics/processors/FunctionCountProcessorSpec.kt Fixed
Comment thread detekt-metrics/src/test/kotlin/dev/detekt/metrics/processors/FunctionCountProcessorSpec.kt Fixed
Comment thread detekt-metrics/src/test/kotlin/dev/detekt/metrics/processors/FunctionCountProcessorSpec.kt Fixed
@BraisGabin BraisGabin force-pushed the optimize-metrics-statistics branch from b1d585a to 28338d0 Compare April 1, 2026 11:08
@BraisGabin BraisGabin force-pushed the optimize-metrics-statistics branch from 28338d0 to 0b861db Compare April 1, 2026 13:08
@BraisGabin BraisGabin mentioned this pull request Apr 1, 2026
Open
@schalkms schalkms added this to the 2.0.0 milestone Apr 4, 2026
@schalkms schalkms merged commit ba8bba7 into main Apr 4, 2026
21 checks passed
@schalkms schalkms deleted the optimize-metrics-statistics branch April 4, 2026 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@schalkms schalkms schalkms approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

2.0.0

Development

Successfully merging this pull request may close these issues.

3 participants

@BraisGabin @schalkms @github-advanced-security