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

Add improved debugging configuration #35

Merged
merged 8 commits into from
Feb 3, 2024
Merged

Add improved debugging configuration #35

merged 8 commits into from
Feb 3, 2024

Conversation

MillironX
Copy link
Member

Some nice-to-haves for debugging the action locally. To be merged only after #34.

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.
This feature is needed by the VSCode debugger.
@MillironX MillironX merged commit 791b428 into nf-core:master Feb 3, 2024
14 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.

1 participant