Closed muffato closed 3 days ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit aef0532
+| ✅ 133 tests passed |+
#| ❔ 24 tests were ignored |#
!| ❗ 1 tests had warnings |!
It's ready for review
I can see master
still being used in this ci?
https://github.com/sanger-tol/blobtoolkit/blob/dev/.github/workflows/branch.yml
if you haven't changed anywhere else, I can do a quick PR.
I can see
master
still being used in this ci? https://github.com/sanger-tol/blobtoolkit/blob/dev/.github/workflows/branch.ymlif you haven't changed anywhere else, I can do a quick PR.
Yes, please
I have successfully run both test
and test_raw
profile on the farm as a new users.
Except the error with the two newly pulled singularity images:
docker.io-pacificbiosciences-pyyaml-5.3.1.img
docker.io-genomehubs-blobtk-0.5.1.img
And the error message:
Command error:
FATAL: container creation failed: mount /proc/self/fd/12->/software/singularity/3.11.4/var/singularity/mnt/session/rootfs error: while mounting image /proc/self/fd/12: kernel reported a bad superblock for squashfs image partition, possible causes are that your kernel doesn't support the compression algorithm or the image is corrupted
I have tried different version of singualrity on the new farm, all have the same problem. This is probably caused by a combination of farm22 and docker image?
Since #97 is taking a while, I've rebased this branch onto dev
black
) is failingTo keep the code consistent with lots of contributors, we run automated code consistency checks. To fix this CI test, please run:
black
: pip install black
black .
Once you push these changes the test should pass, and you can hide this comment :+1:
We highly recommend setting up Black in your code editor so that this formatting is done automatically on save. Ask about it on Slack for help!
Thanks again for your contribution!
TOLIT-2200
This PR is to publish more output files from the pipeline. Those things were not originally included because we don't need them ourselves (we'll get the files via other pipelines) but someone running the pipeline may find those interesting.
At the same time, I found that the full-test didn't pass because of some recent changes I did.
Keeping it as a draft for now until the base branch is merged. I also need to document the changes in
docs/output.md
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).