Repository navigation
Make most pipeline nf-tests executable with any WEBIN credentials - #116
Conversation
|
❌ nf-test failed with latest Nextflow versionNote Tests with Nextflow's latest version failed but it will not cause a CI workflow failure.
See the full run for details. |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
8bdec9b to
c5301db
Compare
06af0c7 to
3ffd6a5
Compare
KateSakharova
left a comment
There was a problem hiding this comment.
Have you checked with Maira about that strategy? I feel like creating SO many studies in ENA might not be the best idea. You can imagine how many times it would be done - every commit in all branches for almost all tests. I think it is VERY A LOT...
I like idea of not using pre-registered study but I also think that approach might not be the best.
I currently don't have better solution. Maybe we should turn off some tests in GA for feature branches and leave those in dev and main only? But that will require carefully running all tests locally which is pain ...
|
@KateSakharova good point... I've asked Maira about it. It's a pity we haven't thought about it earlier. So much work done already 😢 UPD: Maira replied that it's fine until we submit to the TEST server |
KateSakharova
left a comment
There was a problem hiding this comment.
Considering ENA is happy with many studies - let it be!
Resolves #95
Resolves #113
PR checklist
nf-core pipelines lint).nextflow run . -profile test,docker --outdir <OUTDIR>).nextflow run . -profile debug,test,docker --outdir <OUTDIR>).docs/usage.mdis updated.docs/output.mdis updated.CHANGELOG.mdis updated.README.mdis updated (including new tool citations and authors/contributors).