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

Isolate scheduled tests #186

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

darrylmelander
Copy link
Collaborator

GitHub disables workflows with a schedule trigger if you go too long without making changes. When the workflow is disabled, it appears to disable all triggers, not just the schedule trigger. This PR is an attempt to avoid that by splitting the CI workflow into two, one for scheduled triggers and one for everything else.

Github disables workflows with a cron schedule if the repo goes too long
without any changes. It appears to disable all triggers, not just the
schedule trigger. As a result, PRs never get tested and sit there in limbo
forever. This is a) a test to see if that is really what is causing our
PR tests to never run, and b) a fix for the issue.

The fix is to split testing into two separate actions, one for scheduled
tests and one for all other tests. Both call the run-tests workflow.
@darrylmelander darrylmelander force-pushed the isolate-scheduled-tests branch from bce45fe to ef95e18 Compare January 2, 2024 23:09
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