Skip to content

Linting: add linting of nf-test files content#3580

Merged
mirpedrol merged 15 commits into
nf-core:devfrom
mirpedrol:pipeline-nf-test-linting
May 28, 2025
Merged

Linting: add linting of nf-test files content#3580
mirpedrol merged 15 commits into
nf-core:devfrom
mirpedrol:pipeline-nf-test-linting

Conversation

@mirpedrol

@mirpedrol mirpedrol commented May 22, 2025

Copy link
Copy Markdown
Member

Add new lint test nf_test_content to validate the content of nf-test files and associated configuration files in Nextflow pipelines.
close #3065 #3064 #3063

missing: #3049

Comment thread nf_core/pipelines/lint/nf_test_content.py Outdated
Comment thread nf_core/pipelines/lint/nf_test_content.py Outdated
Comment thread nf_core/pipelines/lint/nf_test_content.py Outdated
@mashehu mashehu changed the title Template: add linting of nf-test files content Linting: add linting of nf-test files content May 26, 2025
Comment thread nf_core/pipelines/lint/nf_test_content.py Outdated
@mashehu

mashehu commented May 27, 2025

Copy link
Copy Markdown
Contributor

@nf-core-bot changelog

Comment thread nf_core/pipelines/lint/nf_test_content.py Outdated
@codecov

codecov Bot commented May 27, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.50%. Comparing base (5e25e56) to head (1ade253).
Report is 12 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.

Comment thread nf_core/pipelines/lint/nf_test_content.py Outdated
Comment thread nf_core/pipelines/lint/nf_test_content.py Outdated
Comment thread tests/pipelines/test_lint.py Outdated
Comment thread nf_core/utils.py
Comment thread nf_core/pipelines/lint/nf_test_content.py Outdated
Comment thread nf_core/pipelines/lint/nf_test_content.py Outdated
Comment thread nf_core/pipelines/lint/nf_test_content.py Outdated
@mashehu

mashehu commented May 27, 2025

Copy link
Copy Markdown
Contributor

@nf-core-bot fix linting

@mirpedrol mirpedrol enabled auto-merge May 28, 2025 09:20
@mirpedrol mirpedrol merged commit c08db2d into nf-core:dev May 28, 2025
96 of 97 checks passed
@mirpedrol mirpedrol deleted the pipeline-nf-test-linting branch May 28, 2025 09:33
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.

4 participants