Skip to content

Conversation

@simonw
Copy link
Owner

@simonw simonw commented Dec 12, 2025

@simonw simonw added the ci label Dec 12, 2025
@codecov
Copy link

codecov bot commented Dec 12, 2025

Codecov Report

❌ Patch coverage is 91.42857% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 95.20%. Comparing base (066d0f3) to head (50a979b).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
sqlite_utils/cli.py 92.77% 6 Missing ⚠️
sqlite_utils/db.py 50.00% 2 Missing ⚠️
sqlite_utils/utils.py 94.44% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #693      +/-   ##
==========================================
- Coverage   95.34%   95.20%   -0.14%     
==========================================
  Files           8        8              
  Lines        3007     3088      +81     
==========================================
+ Hits         2867     2940      +73     
- Misses        140      148       +8     

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

@simonw
Copy link
Owner Author

simonw commented Dec 12, 2025

OK wow, Claude Code just crunched through the rest of the warnings as well: https://gistpreview.github.io/?0bb8e869b82f6ff0db647de755182502

@simonw simonw marked this pull request as ready for review December 12, 2025 00:46
@simonw simonw merged commit fd5b09f into main Dec 12, 2025
90 checks passed
@simonw simonw deleted the warnings branch December 12, 2025 00:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants