nf-core / eager

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

Patch 2.5.1 #1052

Closed TCLamnidis closed 4 months ago

TCLamnidis commented 4 months ago

Patch release 2.5.1 - Bopfingen

Closes #1036 Closes #1037 Closes #1047 Closes #1048 Closes #1049 Closes #1051

PR checklist

github-actions[bot] commented 4 months ago

This PR is against the master branch :x:


Hi @TCLamnidis,

It looks like this pull-request is has been made against the nf-core/eager master branch. The master branch on nf-core repositories should always contain code from the latest release. Because of this, PRs to master are only allowed if they come from the nf-core/eager dev branch.

You do not need to close this PR, you can change the target branch to dev by clicking the "Edit" button at the top of this page. Note that even after this, the test will continue to show as failing until you push a new commit.

Thanks again for your contribution!

github-actions[bot] commented 4 months 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 4 months ago

All markdown linting fails for various reasons, but linting is fine locally (except for logos being outdated).

TCLamnidis commented 4 months ago

I will update the date in the changelog and make the release tomorrow