nf-core / eager

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

2.4.7 Patch #984

Closed jfy133 closed 1 year ago

jfy133 commented 1 year ago

Closes https://github.com/nf-core/eager/issues/983 and a broken conda environment

Tested by manually editing environment to bump the pygments to the minimal version as reported in teh error, and then linted with nf-core tools 1.9

PR checklist

github-actions[bot] commented 1 year ago

Markdown linting is failing

To keep the code consistent with lots of contributors, we run automated code consistency checks. To fix this CI test, please run:

Once you push these changes the test should pass, and you can hide this comment :+1:

We highly recommend setting up markdownlint 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!

TCLamnidis commented 1 year ago

@aidaanva was adamant that we should also bump MultiQC to fix a division by 0 error. Is there a reason why we don't? and if we do, maybe double check that the dependencies work with it 👀