Skip to content

fix: handle missing Content-Length header, fix docstring param name

0ea6f98
Select commit
Loading
Failed to load commit list.
Merged

fix: use FileLock with atomic rename for safe concurrent model downloads #396

fix: handle missing Content-Length header, fix docstring param name
0ea6f98
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Feb 20, 2026 in 1s

94.87% (+0.88%) compared to 06c30cd

View this Pull Request on Codecov

94.87% (+0.88%) compared to 06c30cd

Details

Codecov Report

❌ Patch coverage is 96.13260% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.87%. Comparing base (06c30cd) to head (0ea6f98).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
tests_unit/test_fetch_model_file.py 97.05% 4 Missing ⚠️
nc_py_api/ex_app/integration_fastapi.py 93.33% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #396      +/-   ##
==========================================
+ Coverage   93.99%   94.87%   +0.88%     
==========================================
  Files          45       46       +1     
  Lines        5392     5540     +148     
==========================================
+ Hits         5068     5256     +188     
+ Misses        324      284      -40     
Files with missing lines Coverage Δ
nc_py_api/ex_app/integration_fastapi.py 52.48% <93.33%> (+29.40%) ⬆️
tests_unit/test_fetch_model_file.py 97.05% <97.05%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.