Skip to content

feat(pull): check all files were downloaded#134

Merged
tjzegmott merged 3 commits intomainfrom
pull-ensure-all-files-downloaded
Feb 19, 2026
Merged

feat(pull): check all files were downloaded#134
tjzegmott merged 3 commits intomainfrom
pull-ensure-all-files-downloaded

Conversation

@tjzegmott
Copy link
Contributor

@tjzegmott tjzegmott commented Feb 19, 2026

Non-zero exit code in case of an error. This should show as a failed step in Workflow.

@coveralls
Copy link

coveralls commented Feb 19, 2026

Coverage Status

coverage: 66.489% (+0.5%) from 66.0%
when pulling 2c5a4c9 on pull-ensure-all-files-downloaded
into 8b35e4f on main.

@tjzegmott tjzegmott force-pushed the pull-ensure-all-files-downloaded branch from d17b177 to 6b6fbfa Compare February 19, 2026 20:44
@tjzegmott tjzegmott force-pushed the pull-ensure-all-files-downloaded branch from 18e423e to 2c5a4c9 Compare February 19, 2026 23:13
@tjzegmott tjzegmott merged commit eea76ef into main Feb 19, 2026
6 checks passed
@tjzegmott tjzegmott deleted the pull-ensure-all-files-downloaded branch February 19, 2026 23:26
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 80.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.48%. Comparing base (8b35e4f) to head (2c5a4c9).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
dtcli/pull.py 50.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #134      +/-   ##
==========================================
+ Coverage   66.00%   66.48%   +0.48%     
==========================================
  Files          14       14              
  Lines        1300     1313      +13     
==========================================
+ Hits          858      873      +15     
+ Misses        442      440       -2     

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

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.

3 participants