Skip to content

Code stats - #119

Merged
jayvdb merged 2 commits into
mainfrom
sloc
Aug 20, 2026
Merged

Code stats#119
jayvdb merged 2 commits into
mainfrom
sloc

Conversation

@jayvdb

@jayvdb jayvdb commented Aug 19, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Chores
    • Updated automated build, test, coverage, dependency, and security workflows for improved reliability.
    • Added source-code line-count reporting with validation and downloadable reports.
    • Improved .NET WebAssembly setup by installing required tooling once and coordinating build steps.
    • Streamlined .NET dependency prefetching to use restore-only behavior.

@coderabbitai

coderabbitai Bot commented Aug 19, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 654f026c-0b6b-41da-9af6-e3582c38d5ae

📥 Commits

Reviewing files that changed from the base of the PR and between c38948b and 6cd92b7.

⛔ Files ignored due to path filters (1)
  • .mise/mise.coverage.lock is excluded by !**/*.lock
📒 Files selected for processing (7)
  • .github/workflows/check.yaml
  • .github/workflows/codeql.yaml
  • .github/workflows/coverage.yaml
  • .github/workflows/dependencies.yaml
  • .github/workflows/test.yaml
  • .mise/config.coverage.toml
  • .mise/config.dotnet.toml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

CI workflows update action versions and add SLOC reporting. Mise configuration adds SLOC counting and cross-check tasks. .NET tasks centralize WASM workload installation and order related builds.

Changes

CI tooling

Layer / File(s) Summary
GitHub Actions version updates
.github/workflows/check.yaml, .github/workflows/codeql.yaml, .github/workflows/dependencies.yaml, .github/workflows/test.yaml
Checkout, cache, and CodeQL reference versions are updated across CI workflows.
SLOC reporting and coverage integration
.mise/config.coverage.toml, .github/workflows/coverage.yaml
The new sloc and sloc-crosscheck tasks count and compare tracked-file partitions. The coverage workflow publishes the report and uploads SLOC output artifacts while updating coverage action versions.

.NET workflow orchestration

Layer / File(s) Summary
Shared WASM workload and build scheduling
.mise/config.dotnet.toml
A shared task installs wasm-tools. WASM builds depend on it, math waits for the data build when both run, and prefetch performs dotnet restore after workload setup.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: ⚪ Minimal · up to 6cd92

This change introduces no actionable merge-blocking risk beyond normal checks and review.

Sequence Diagram(s)

sequenceDiagram
  participant CoverageWorkflow
  participant MiseTasks
  participant SCC
  participant Gocloc
  participant GitHubSummary
  CoverageWorkflow->>MiseTasks: run sloc-crosscheck
  MiseTasks->>SCC: count tracked-file partitions
  MiseTasks->>Gocloc: recount partitions
  Gocloc-->>MiseTasks: return counts and deltas
  MiseTasks-->>CoverageWorkflow: return cross-check report
  CoverageWorkflow->>GitHubSummary: append report
Loading

Suggested reviewers: pierre-tenedero

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title identifies the new code-statistics functionality, which is a significant part of the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch sloc

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@deepsource-io

deepsource-io Bot commented Aug 19, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in c38948b...6cd92b7 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Coverage  

Code Review Summary

Analyzer Status Updated (UTC) Details
C# Aug 19, 2026 9:54a.m. Review ↗
C & C++ Aug 19, 2026 9:54a.m. Review ↗
Docker Aug 19, 2026 9:54a.m. Review ↗
Java Aug 19, 2026 9:54a.m. Review ↗
JavaScript Aug 19, 2026 9:54a.m. Review ↗
Python Aug 19, 2026 9:54a.m. Review ↗
Rust Aug 19, 2026 9:54a.m. Review ↗
Secrets Aug 19, 2026 9:54a.m. Review ↗
Code coverage Aug 19, 2026 10:19a.m. Review ↗

Code Coverage Summary

Language Line Coverage (Overall)
Aggregate
58.7%
Python
89.6%
Rust
56%

➟ Additional coverage metrics may have been reported. See full coverage report ↗


Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@codecov

codecov Bot commented Aug 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@jayvdb
jayvdb marked this pull request as ready for review August 19, 2026 13:39
@jayvdb
jayvdb requested a review from pierre-tenedero August 19, 2026 13:39
@jayvdb
jayvdb merged commit 89da19f into main Aug 20, 2026
35 checks passed
@jayvdb
jayvdb deleted the sloc branch August 20, 2026 03:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants