log(BREAKING!): replace tableau logger with an interface which zap.SugaredLogger implements#347
Open
Kybxd wants to merge 3 commits into
Open
log(BREAKING!): replace tableau logger with an interface which zap.SugaredLogger implements#347Kybxd wants to merge 3 commits into
zap.SugaredLogger implements#347Kybxd wants to merge 3 commits into
Codecov / codecov/project
succeeded
Feb 6, 2026 in 0s
76.44% (target 63.00%)
View this Pull Request on Codecov
76.44% (target 63.00%)
Details
Codecov Report
❌ Patch coverage is 66.66667% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.44%. Comparing base (1da426f) to head (a64463a).
| Files with missing lines | Patch % | Lines |
|---|---|---|
| internal/importer/excel.go | 33.33% | 1 Missing and 1 partial |
| log/driver/zapdriver/logger.go | 33.33% | 2 Missing |
| cmd/tableauc/main.go | 0.00% | 1 Missing |
Additional details and impacted files
@@ Coverage Diff @@
## master #347 +/- ##
==========================================
+ Coverage 75.12% 76.44% +1.32%
==========================================
Files 85 79 -6
Lines 8571 8233 -338
==========================================
- Hits 6439 6294 -145
+ Misses 1565 1396 -169
+ Partials 567 543 -24 ☔ 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.
Loading