csawye01 / nf-core-demultiplex

Demultiplexing pipeline for sequencing data
MIT License
0 stars 1 forks source link

Update template code and first pass corrections #1

Closed drpatelh closed 5 years ago

drpatelh commented 5 years ago

Many thanks to contributing to nf-core/demultiplex!

Please fill in the appropriate checklist below (delete whatever is not relevant). These are the most common things requested on pull requests (PRs).

PR checklist

Learn more about contributing: https://github.com/nf-core/demultiplex/tree/master/.github/CONTRIBUTING.md

Hi @csawye01 !

So this is a first pass PR that updates the template code for the pipeline to version 1.6. We will need to do this at some point anyway for the creation of the TEMPLATE branch for the automated template code syncing: https://nf-co.re/developers/sync

I havent looked at the processes themselves but we can address these changes in another PR.

Have a look at the changes and make any comments/suggestions if you want me to change anything. Then when you are happy with the changes just merge the pull request. Once merged the pipeline will require testing because I have no way of using automated tests at the moment until we set-up a test-dataset.