Skip to content

Comments

File Logger Refactor#17

Merged
treyburn merged 14 commits intomainfrom
treyburn/file-logger-refactor
Oct 31, 2025
Merged

File Logger Refactor#17
treyburn merged 14 commits intomainfrom
treyburn/file-logger-refactor

Conversation

@treyburn
Copy link
Owner

@treyburn treyburn commented Oct 24, 2025

  • Restructured/renamed the logger package
  • Fixed concurrency bugs in the File transaction logger
  • Added unit tests for the File transaction logger

Unrelated:

  • Added formatter configurations for import ordering
  • Skip aggressive code quality lints in tests

@treyburn treyburn self-assigned this Oct 24, 2025
@codecov
Copy link

codecov bot commented Oct 31, 2025

Codecov Report

❌ Patch coverage is 54.54545% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 32.68%. Comparing base (07f9a71) to head (d797773).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
cmd/api/main.go 0.00% 10 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main      #17       +/-   ##
===========================================
+ Coverage   14.67%   32.68%   +18.01%     
===========================================
  Files           5        5               
  Lines         334      257       -77     
===========================================
+ Hits           49       84       +35     
+ Misses        283      170      -113     
- Partials        2        3        +1     

☔ 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.

@treyburn treyburn merged commit f6428ab into main Oct 31, 2025
5 checks passed
@treyburn treyburn deleted the treyburn/file-logger-refactor branch October 31, 2025 17:03
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