nf-core / eager

A fully reproducible and state-of-the-art ancient DNA analysis pipeline
https://nf-co.re/eager
MIT License
146 stars 82 forks source link

2.1 update clean up notes #359

Closed jfy133 closed 4 years ago

jfy133 commented 4 years ago

Authors

MultiQC

Main.nf

Tests

get_software_versions

PR Template

apeltzer commented 4 years ago

MultiQC is up to you now ;-)

Main.nf example command update too - once we have that, we can release I guess.

apeltzer commented 4 years ago

Readme has full list now. Contributors are sorted alphabetically now too.

The manifest file is too long for tower.nf if we keep it complete - I replaced it there with "the nf-core/eager community", same in the dockerfile. The main.nf refers to the readme on the webpage now too - I think this is fine for everyone 👍

jfy133 commented 4 years ago

Will do remaining points, and will split authors Vs contributors - as discussed on slack.

Will also try get GATK 3.5 GitHub actions test to work but not important.

jfy133 commented 4 years ago

Going to leave " Add more File Cleanup Strings to MultiQC yaml to have samples merged if possible" until 2.2

Will do a big overhaul of MultiQC stuff when 1.9 comes out as that will bring a lot of waited-upon functionality. TSV input will also make this much easier to standardise what we clip off.