Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Important! Template update for nf-core/tools v2.7.1 #42

Merged
merged 6 commits into from
Dec 11, 2022

Conversation

nf-core-bot
Copy link
Member

Version 2.7.1 of nf-core/tools has just been released with updates to the nf-core template. This automated pull-request attempts to apply the relevant updates to this pipeline.

Please make sure to merge this pull-request as soon as possible, resolving any merge conflicts in the nf-core-template-merge-2.7.1 branch (or your own fork, if you prefer). Once complete, make a new minor release of your pipeline.

For instructions on how to merge this PR, please see https://nf-co.re/developers/sync.

For more information about this release of nf-core/tools, please see the v2.7.1 release page.

@github-actions
Copy link

github-actions bot commented Dec 9, 2022

Python linting (black) is failing

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

  • Install black: pip install black
  • Fix formatting errors in your pipeline: black .

Once you push these changes the test should pass, and you can hide this comment 👍

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

@edmundmiller edmundmiller added this to the v1.0.0 milestone Dec 10, 2022
@edmundmiller edmundmiller self-requested a review December 10, 2022 13:01
@edmundmiller
Copy link
Collaborator

nf-core/tools#2121 This is why the nf-core lint is failing

@alyssa-ab alyssa-ab self-requested a review December 10, 2022 21:28
Copy link
Collaborator

@alyssa-ab alyssa-ab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fixes!

@alyssa-ab alyssa-ab merged commit a6285ad into dev Dec 11, 2022
@edmundmiller edmundmiller deleted the nf-core-template-merge-2.7.1 branch December 11, 2022 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants