Closed nhhaidee closed 1 year ago
Can you change the PR target to CFIA-NCFAD?
To keep the code consistent with lots of contributors, we run automated code consistency checks. To fix this CI test, please run:
yaml-lint
npm
then install yaml-lint
(npm install -g yaml-lint
)yamllint $(find . -type f -name "*.yml" -o -name "*.yaml")
Once you push these changes the test should pass, and you can hide this comment :+1:
We highly recommend setting up yaml-lint 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!
Add
flags=re.IGNORECASE
to handle subtype in lower case because there are 32 sequences in NCBI database have subtype in lower casePR checklist
scrape_software_versions.py
nf-core lint .
).nextflow run . -profile test,docker
).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).