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

ci: Update CI workflows to be compatible with act #34

Merged
merged 4 commits into from
Feb 3, 2024
Merged

ci: Update CI workflows to be compatible with act #34

merged 4 commits into from
Feb 3, 2024

Conversation

MillironX
Copy link
Member

I've used act for a while now, and it's become more and more indispensable as I dive into the deep bowls of Github Actions. I got tired of constantly git stash and git stash poping these changes, and thought they would be useful for others.

We have two CI workflows with the name "test" (three if you count
"test-14"). This ambiguates the job name, and results in act running both
unit tests and integration tests when running `act -j test`. Fix that by
changing the names.
For some reason, I need to pin the Java install when using act. This is not
needed when using the GitHub Actions runner, so add an escape clause to
only run this extra step when using act.
@MillironX MillironX merged commit a46a611 into nf-core:master Feb 3, 2024
16 checks passed
@MillironX MillironX deleted the feature/act-friendly branch February 3, 2024 17:16
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