search

datalad / datalad-installer

Installation script for Datalad and related components
MIT License
6 stars 3 forks source link

Handle lack of Link header when paginating #188

Closed jwodder closed 9 months ago

codecov-commenter commented 9 months ago

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (261568b) 65.44% compared to head (db3dae6) 65.32%.

Files Patch % Lines
src/datalad_installer.py 0.00% 4 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #188 +/- ## ========================================== - Coverage 65.44% 65.32% -0.12% ========================================== Files 1 1 Lines 1641 1644 +3 Branches 407 408 +1 ========================================== Hits 1074 1074 - Misses 465 468 +3 Partials 102 102 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

yarikoptic commented 9 months ago

Just for the reference -- where did you run into this case when you haven't ran into it before?

jwodder commented 9 months ago

@yarikoptic A scheduled test run last night failed because of this. I don't know how that happened, as all the paginated requests the script makes should be to things that always have Link headers.