Skip to content

Only run 'download' actions once core completes + re-add ubuntu-latest to matrix #276

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

Merged
merged 4 commits into from
Mar 26, 2025

Conversation

pjrobertson
Copy link
Collaborator

Now that the timestamping enricher is merged, we can now re-enable ubuntu-latest in the testing matrix.

Also, to cut back on the download tests, I've set it to only run once the core tests succeed.

@pjrobertson
Copy link
Collaborator Author

P.S. I pushed one small code change here just to trigger the tests.

paths:
- src/**

jobs:
tests:
# Only run if the triggering workflow (tests-core) succeeded
if: ${{ github.event_name == 'schedule' || github.event.workflow_run.conclusion == 'success' }}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Good plan!

# clear out any empty strings that a user may have erroneously added
continue
if not check(dep):
for dep in filter(lambda d: len(d.strip()), deps):
Copy link
Collaborator

Choose a reason for hiding this comment

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

good catch!

@pjrobertson pjrobertson merged commit fbf51f6 into main Mar 26, 2025
7 checks passed
@pjrobertson pjrobertson deleted the actions_updates branch March 26, 2025 19: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.

2 participants