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 gantt plot nf-test #13

Merged
merged 12 commits into from
Nov 30, 2023
Merged

Add gantt plot nf-test #13

merged 12 commits into from
Nov 30, 2023

Conversation

pinin4fjords
Copy link
Member

@pinin4fjords pinin4fjords commented Nov 28, 2023

Set up tests for the gantt plots, chaining from the dump

Base automatically changed from nf-test to main November 29, 2023 14:47
@pinin4fjords pinin4fjords changed the title Start with tests for the gantt plots Add gantt plot nf-test Nov 29, 2023
@@ -0,0 +1,2 @@
process { withName: 'SEQERA_RUNS_DUMP' { container = 'public.ecr.aws/seqera-labs/wave/containers:tower-cli-0.9.0--2cb0f2e9d85d026b' } }
Copy link
Member

Choose a reason for hiding this comment

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

Why do we need this file? Because I added a separate container.config? Maybe we need to put this in the process for now and we can move it later.

Copy link
Member Author

Choose a reason for hiding this comment

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

For some reason the global container config wasn't pulled into the the tests. I tried using projectDir on the config line where you included it, but that didn't do the trick. So I put it in the 'figure out later' box.

Copy link
Member

Choose a reason for hiding this comment

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

Ok. I'll have a play locally. Thanks!

@drpatelh drpatelh merged commit 4deb372 into main Nov 30, 2023
9 checks passed
@drpatelh drpatelh deleted the gantt_nftests branch December 13, 2023 19:51
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