Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(ci): Increase test timeout to 30m (~2x median) #469

Merged
merged 1 commit into from
Mar 7, 2025

Conversation

effigies
Copy link
Member

@effigies effigies commented Mar 6, 2025

Hitting 20m timeout. Most runs finish in the 16-17m range, so 30m is a safer indicator that something is actually wrong.

Copy link

codecov bot commented Mar 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.19%. Comparing base (1f16768) to head (bcbaa19).
Report is 2 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master     #469       +/-   ##
===========================================
+ Coverage   63.23%   74.19%   +10.96%     
===========================================
  Files          25       25               
  Lines        1961     1957        -4     
  Branches      246      245        -1     
===========================================
+ Hits         1240     1452      +212     
+ Misses        683      438      -245     
- Partials       38       67       +29     
Flag Coverage Δ
ds054 45.59% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@effigies effigies force-pushed the ci/increase-timeout branch from ad7e94b to bcbaa19 Compare March 6, 2025 17:09
@effigies effigies merged commit ba657db into nipreps:master Mar 7, 2025
21 checks passed
@effigies effigies deleted the ci/increase-timeout branch March 7, 2025 00:51
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.

1 participant