Skip to content

ci: disable scheduled test-live runs (manual dispatch only)#246

Merged
kyle-messier merged 1 commit into
mainfrom
fix/coverage-timeout-spei-smoketest
May 27, 2026
Merged

ci: disable scheduled test-live runs (manual dispatch only)#246
kyle-messier merged 1 commit into
mainfrom
fix/coverage-timeout-spei-smoketest

Conversation

@kyle-messier
Copy link
Copy Markdown
Member

The GitHub-hosted runner network is too unreliable for the multi-GB science-data downloads these tests perform (SPEI, MERRA-2, etc.), producing constant false-positive failure issues. Run live tests locally instead. workflow_dispatch is retained for on-demand runs.

The GitHub-hosted runner network is too unreliable for the multi-GB
science-data downloads these tests perform (SPEI, MERRA-2, etc.),
producing constant false-positive failure issues. Run live tests
locally instead. workflow_dispatch is retained for on-demand runs.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 27, 2026 16:52
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Disables the nightly cron trigger for the live-API test workflow, leaving only manual workflow_dispatch runs, because GitHub-hosted runners are too unreliable for the multi-GB downloads performed by SPEI, MERRA-2, and similar live tests.

Changes:

  • Remove the schedule: cron '0 6 * * *' trigger from test-live.yaml.
  • Rewrite the header comment to explain the rationale and direct contributors to run live tests locally.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +7 to +8
# `vignettes/testing.Rmd` ("Running live tests locally") or the README
# section "Live tests".
@kyle-messier kyle-messier merged commit bb0cd5d into main May 27, 2026
10 checks passed
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