Open rraadd88 opened 1 month ago
master
branch :x:base
to dev
Hi @rraadd88,
It looks like this pull-request is has been made against the rraadd88/rnaseq-1 master
branch.
The master
branch on nf-core repositories should always contain code from the latest release.
Because of this, PRs to master
are only allowed if they come from the rraadd88/rnaseq-1 dev
branch.
You do not need to close this PR, you can change the target branch to dev
by clicking the "Edit" button at the top of this page.
Note that even after this, the test will continue to show as failing until you push a new commit.
Thanks again for your contribution!
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit 49da4a8
+| ✅ 173 tests passed |+
#| ❔ 9 tests were ignored |#
!| ❗ 8 tests had warnings |!
This is a suggestion to clarify the description of the
--skip_qc
argument. Based on my understanding of the workflow, this argument applies to the stage 4 (post-processing) and 5 (final QC) including DeSeq based QC. I hope this is correct.Clarification of which QCs are skipped is necessary because steps in stage could also be considered as QC steps. Perhaps a list of skipped QCs would be helpful. For example, I am not sure if
--skip_qc
skipsRSeQC
or not.I hope the documentation for this argument could be more informative.
PR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).nextflow run . -profile debug,test,docker --outdir <OUTDIR>
).docs/usage.md
is updated.docs/output.md
is updated.CHANGELOG.md
is updated.README.md
is updated (including new tool citations and authors/contributors).