Closed sateeshperi closed 1 year ago
master
branch :x:base
to dev
Hi @sateeshperi,
It looks like this pull-request is has been made against the nf-core/ampliseq 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 nf-core/ampliseq 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 6633e73
+| ✅ 155 tests passed |+
#| ❔ 1 tests were ignored |#
!| ❗ 2 tests had warnings |!
Interesting, seems to check for file existence and file md5sums for test results?
that's correct. I try to snapshot (named md5sum json) everything in the output folder for each test profile and do file exists checks for files that can't generate a consistent snapshot. It is still POC but you can give it a try by installing nf-test and running nf-test test
from the pipeline repo.
These can probably be cleaned up with a rebase
PR checklist
nf-core lint
).nextflow run . -profile 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).