Skip to content

toml+log+stats#258

Merged
hugoh merged 4 commits into
mainfrom
toml+log+stats
Jun 4, 2026
Merged

toml+log+stats#258
hugoh merged 4 commits into
mainfrom
toml+log+stats

Conversation

@hugoh

@hugoh hugoh commented Jun 4, 2026

Copy link
Copy Markdown
Owner
  • build: fixed test deps
  • fix(toml): switched parser
  • fix(logging): improved format
  • refactor(stats): improved HTTP server handling

@sonarqubecloud

sonarqubecloud Bot commented Jun 4, 2026

Copy link
Copy Markdown

@deepsource-io

deepsource-io Bot commented Jun 4, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 2af13d4...ea0ef59 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  

Code Review Summary

Analyzer Status Updated (UTC) Details
Go Jun 4, 2026 11:00p.m. Review ↗
Secrets Jun 4, 2026 11:00p.m. Review ↗

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.

@codecov

codecov Bot commented Jun 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 82.75862% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.36%. Comparing base (2af13d4) to head (ea0ef59).

Files with missing lines Patch % Lines
internal/status/stat-server.go 85.00% 3 Missing and 3 partials ⚠️
internal/conf.go 72.72% 3 Missing ⚠️
internal/logic/downaction.go 80.00% 3 Missing ⚠️
internal/logic/loop.go 80.00% 2 Missing ⚠️
internal/cmd.go 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #258      +/-   ##
==========================================
+ Coverage   88.16%   88.36%   +0.19%     
==========================================
  Files          19       18       -1     
  Lines         845      842       -3     
==========================================
- Hits          745      744       -1     
+ Misses         69       67       -2     
  Partials       31       31              

☔ View full report in Codecov by Harness.
📢 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.

@hugoh hugoh merged commit ad309d6 into main Jun 4, 2026
8 of 9 checks passed
@hugoh hugoh deleted the toml+log+stats branch June 4, 2026 23:21
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.

1 participant