Closed taylorreiter closed 1 year ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit 4011cde
+| ✅ 131 tests passed |+
#| ❔ 21 tests were ignored |#
!| ❗ 1 tests had warnings |!
Wait did I miss something re the action composition stuff? does that comment highlight a strategy or is there something more I should be seeing?
in either case, I'll take a look at the nextflow config, thank you for catching this!
re DOI/version -- those can't be updated until after the first release when zenodo assigns the pipeline a DOI.
@taylorreiter I added a comment about it inline. This is the relevant strategy for deduping: https://docs.github.com/en/actions/creating-actions/creating-a-composite-action
I think the profile template conf is here: https://github.com/nf-core/configs/blob/master/conf/aws_tower.config
The goal of this PR is to set up full tests that will run automatically when there's a new release. Through secrets I set up on the repo, this should automagically launch on our tower account and dump the results into the seqqc s3 bucket. The test data sets are in Arcadia-Science/test-datasets already, and I have already tested them on Tower and they run with the current
dev
branch of seqqc. Doing all of these chemistries will also mean that we produce example multiqc reports every time we make a new release, which I think is a good idea.The only thing I'm not sure about is whether each data set should have its own workflow. It made sense to me that it would, but i'm not sure.