Skip to content

fix: Add timeout to avoid resource leak, retry failed fetches#351

Merged
rwblair merged 4 commits intobids-standard:mainfrom
effigies:feat/timeout-retry
Feb 2, 2026
Merged

fix: Add timeout to avoid resource leak, retry failed fetches#351
rwblair merged 4 commits intobids-standard:mainfrom
effigies:feat/timeout-retry

Conversation

@effigies
Copy link
Contributor

@effigies effigies commented Feb 2, 2026

Builds on #349.

@codecov
Copy link

codecov bot commented Feb 2, 2026

Codecov Report

❌ Patch coverage is 86.84211% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.85%. Comparing base (acbd58c) to head (8936c30).

Files with missing lines Patch % Lines
src/files/openers.ts 87.50% 4 Missing ⚠️
src/files/access.ts 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #351      +/-   ##
==========================================
- Coverage   87.02%   86.85%   -0.18%     
==========================================
  Files          51       51              
  Lines        3892     3924      +32     
  Branches      624      622       -2     
==========================================
+ Hits         3387     3408      +21     
- Misses        495      501       +6     
- Partials       10       15       +5     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@effigies effigies marked this pull request as ready for review February 2, 2026 19:57
@rwblair rwblair merged commit 6890959 into bids-standard:main Feb 2, 2026
8 of 23 checks passed
@effigies effigies deleted the feat/timeout-retry branch February 2, 2026 21:45
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.

2 participants