Closed ksenia-krasheninnikova closed 9 months ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit 84bd862
+| ✅ 129 tests passed |+
#| ❔ 19 tests were ignored |#
!| ❗ 5 tests had warnings |!
The new location should be fine for user toldev
and other tol users to run the full tests.
There is one other file still in your person location, is it being used? https://github.com/sanger-tol/genomeassembly/blob/1aa5c6007e18047b6af25fb24821edfe51d5145b/assets/test_iyVesGerm1.yaml#L4
@sanger-tolsoft fix linting
@gq1 I copied the datasets for iyVesGerm1 and gfLaeSulp1 to the new location. They are both of a moderate size may be useful to have an extra dataset for testing ready for the tower. Removed an extra test file as there is no need to keep it on github.
Do you want to merge now? And hope this time it will pass.
And now all your CI tests running on the large runner and it seems fine every time. Hope you can make theorganelles_on
work there next time.
Merged into dev. Thanks @gq1
Update the data location to make it accessible from nextflow tower
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).