Skip to content

fix some nextflow lint warnings in different skipped template versions#4191

Merged
mashehu merged 11 commits intonf-core:devfrom
mashehu:fix-nextflow-lint-warnings-in-template
Apr 22, 2026
Merged

fix some nextflow lint warnings in different skipped template versions#4191
mashehu merged 11 commits intonf-core:devfrom
mashehu:fix-nextflow-lint-warnings-in-template

Conversation

@mashehu
Copy link
Copy Markdown
Contributor

@mashehu mashehu commented Apr 16, 2026

No description provided.

@mashehu mashehu force-pushed the fix-nextflow-lint-warnings-in-template branch from 8b03840 to 0519766 Compare April 16, 2026 19:05
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.13%. Comparing base (5fd422e) to head (14c6388).
⚠️ Report is 24 commits behind head on dev.

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

@mashehu mashehu force-pushed the fix-nextflow-lint-warnings-in-template branch from 0519766 to 8385ca6 Compare April 17, 2026 14:34
@mashehu mashehu force-pushed the fix-nextflow-lint-warnings-in-template branch from 8385ca6 to 6b81ece Compare April 17, 2026 14:45
@mashehu mashehu force-pushed the fix-nextflow-lint-warnings-in-template branch from 2dd5d1f to eef5708 Compare April 20, 2026 13:51
@mashehu mashehu added this to the 4.0.0 milestone Apr 21, 2026
@mashehu mashehu force-pushed the fix-nextflow-lint-warnings-in-template branch from 183703c to a27db09 Compare April 21, 2026 15:35
@mashehu mashehu force-pushed the fix-nextflow-lint-warnings-in-template branch from f51772b to 7600f57 Compare April 22, 2026 08:57
@mashehu mashehu marked this pull request as ready for review April 22, 2026 09:17
@mashehu mashehu changed the title add -fail-on-warning step for nextflow linting step fix some nextflow lint warnings in different skipped template versions Apr 22, 2026
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.

Looks sensible

@mashehu mashehu merged commit 0e87eb7 into nf-core:dev Apr 22, 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.

2 participants