-
Notifications
You must be signed in to change notification settings - Fork 33
Important! Template update for nf-core/tools v3.0.2 #421
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
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fairly uncontroversial, but I don't understand the CI errors. Ping me when those are resolved and I'll approve.
The CI errors all come from a (recent) faulty database update for one tool of the AMP workflow described in #423. Thus, only those CI tests without the AMP workflow succeed atm. All others will fail until either
For template update purposes I am not sure if the database issue is required to be solved first? |
@nf-core-bot fix linting |
Current status, waiting for new AMPcombi modules to be merged in from @Darcy220606 so we can get the tests to poass |
Status update, waiting on: |
Version Please use #429 to merge in the new changes from the nf-core template as soon as possible. |
Add ampcombi 2.0.1
…cscan into nf-core-template-merge-3.0.2
@nf-core-bot fix linting |
Version
3.0.2
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-3.0.2
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/docs/contributing/sync/.
For more information about this release of nf-core/tools, please see the
v3.0.2
release page.