Skip to content

add --force parameter to modules remove and subworkflow remove#4213

Merged
mashehu merged 6 commits intonf-core:devfrom
mashehu:add-force-to-remove
Apr 23, 2026
Merged

add --force parameter to modules remove and subworkflow remove#4213
mashehu merged 6 commits intonf-core:devfrom
mashehu:add-force-to-remove

Conversation

@mashehu
Copy link
Copy Markdown
Contributor

@mashehu mashehu commented Apr 23, 2026

We already hint to use it to ignore left over include statements. Now this actually allows that.

Also found a bug where include statements where not found for non-standard workflow directory structures, for example in workflows/rnaseq/main.nf

@mashehu mashehu requested a review from a team April 23, 2026 11:59
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 23, 2026

Codecov Report

❌ Patch coverage is 71.42857% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.14%. Comparing base (0e87eb7) to head (ae7e150).
⚠️ Report is 6 commits behind head on dev.

Files with missing lines Patch % Lines
nf_core/__main__.py 66.66% 2 Missing ⚠️
nf_core/commands_modules.py 50.00% 1 Missing ⚠️
nf_core/commands_subworkflows.py 50.00% 1 Missing ⚠️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Member

@pinin4fjords pinin4fjords left a comment

Choose a reason for hiding this comment

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

just a typo

Comment thread nf_core/__main__.py Outdated
mashehu and others added 3 commits April 23, 2026 15:11
Co-authored-by: Jonathan Manning <pininforthefjords@gmail.com>
@mashehu mashehu enabled auto-merge (squash) April 23, 2026 13:14
@mashehu mashehu merged commit 6ef3182 into nf-core:dev Apr 23, 2026
116 checks passed
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