Skip to content

Add fgumi/extract#11546

Open
atrigila wants to merge 15 commits intonf-core:masterfrom
atrigila:fgumi_add
Open

Add fgumi/extract#11546
atrigila wants to merge 15 commits intonf-core:masterfrom
atrigila:fgumi_add

Conversation

@atrigila
Copy link
Copy Markdown
Contributor

@atrigila atrigila commented May 6, 2026

Comment thread modules/nf-core/fgumi/extract/main.nf
Copy link
Copy Markdown
Contributor

@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.

Oh, ok, my apologies, I thought it was taking ubam.
You can remove the default prefix then, if it isn't going to collide within the module.
You shouldn't need to set prefix in the test at that point though.

@atrigila atrigila requested a review from SPPearce May 7, 2026 13:03
Comment thread modules/nf-core/fgumi/extract/main.nf Outdated
@atrigila atrigila requested a review from SPPearce May 7, 2026 18:57
@atrigila atrigila marked this pull request as ready for review May 8, 2026 10:52
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.

2 participants