Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@nf-core-bot
Copy link
Member

Version 2.9 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.9 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.9 release page.

@github-actions
Copy link

github-actions bot commented Sep 14, 2023

nf-core lint overall result: Passed ✅ ⚠️

Posted for pipeline commit a23a895

+| ✅ 156 tests passed       |+
!| ❗   7 tests had warnings |!
Details

❗ Test warnings:

  • pipeline_todos - TODO string in README.md: Describe the minimum required steps to execute the pipeline, e.g. how to prepare samplesheets.
  • pipeline_todos - TODO string in methods_description_template.yml: #Update the HTML below to your preferred methods description, e.g. add publication citation for this pipeline
  • pipeline_todos - TODO string in awsfulltest.yml: You can customise AWS full pipeline tests as required
  • system_exit - System.exit in WorkflowRnavar.groovy: System.exit(1) [line 24]
  • system_exit - System.exit in WorkflowRnavar.groovy: System.exit(1) [line 29]
  • system_exit - System.exit in WorkflowRnavar.groovy: System.exit(1) [line 34]
  • system_exit - System.exit in WorkflowRnavar.groovy: System.exit(1) [line 39]

✅ Tests passed:

Run details

  • nf-core/tools version 2.9
  • Run at 2023-09-21 07:01:07

Choose a reason for hiding this comment

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

Add stub to this local module?

emit:
bam = ch_bam_index
reports = ch_reports

Choose a reason for hiding this comment

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

Suggested change

bam = SAMTOOLS_SORT.out.bam // channel: [ val(meta), [ bam ] ]
bai = SAMTOOLS_INDEX.out.bai // channel: [ val(meta), [ bai ] ]
csi = SAMTOOLS_INDEX.out.csi // channel: [ val(meta), [ csi ] ]

Choose a reason for hiding this comment

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

Suggested change


emit:
reports

Choose a reason for hiding this comment

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

Suggested change

@maxulysse
Copy link
Member

Thanks a lot @CarsonJM.
I fixed your comments apart from the stub and the code polishing which I'll do in separate PRs.

Copy link

@SPPearce SPPearce left a comment

Choose a reason for hiding this comment

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

Looks good to me, great start Maxime.

@maxulysse maxulysse merged commit 3664dfa into dev Sep 21, 2023
@maxulysse maxulysse deleted the nf-core-template-merge-2.9 branch September 21, 2023 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants