-
Notifications
You must be signed in to change notification settings - Fork 87
DSL2: nf-test test_long
#1158
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
DSL2: nf-test test_long
#1158
Conversation
|
Warning Newer version of the nf-core template is available. Your pipeline is using an old version of the nf-core template: 3.3.2. For more documentation on how to update your pipeline, please see the nf-core documentation and Synchronisation documentation. |
TCLamnidis
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good in general, but ofc we need to get the tests to pass before merging :/
Are the failing tests due to the pending template merge only?
|
It seems so. has no issues with the snaps. nf-core lint locally has no issues as well. And here it fails and stops with the templates: |
|
This reverts commit 06309d1.
Closes #1139
First version, no review needed yet
PR checklist
nf-core lint .).