-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Hap #68
Hap #68
Conversation
deleted spaces
added spacing in the maps
tidy up and styling
add python and pandas version
Hi Yumi. Which profile can I use to test this ? |
Hi Matthieu, the test profile is this one https://github.com/sanger-tol/treeval/blob/hap/assets/full_s3_treeval_test.yaml |
Co-authored-by: Matthieu Muffato <mm49@sanger.ac.uk>
Co-authored-by: Matthieu Muffato <mm49@sanger.ac.uk>
Co-authored-by: Matthieu Muffato <mm49@sanger.ac.uk>
have done all required changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(just pushed two commits directly to fix the alignment (I got the amount of whitespace wrong) and apply prettier to modules.json)
Looks good to me !
Merging hap - Completes Phase 1.5 and #52
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).