Use the org from .nf-core.yml when linting multiqc_config report_comment#3800
Use the org from .nf-core.yml when linting multiqc_config report_comment#3800
Conversation
Follows the same pattern as commit 984398c which fixed manifest.name and manifest.homePage linting to use dynamic org. This allows custom pipelines with different organizations to pass the multiqc_config lint check without needing to add exceptions.
There was a problem hiding this comment.
Pull Request Overview
This PR updates the multiqc_config lint check to dynamically use the organization name from .nf-core.yml configuration instead of hardcoding "nf-core" in report comment URLs. This follows the same pattern established in commit 984398c which fixed similar issues for manifest.name and manifest.homePage linting.
- Adds dynamic organization name resolution from
.nf-core.ymltemplate configuration - Updates both dev and release version report comment URL generation to use the dynamic org name
- Allows custom pipelines with different organizations to pass the multiqc_config lint check
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
mirpedrol
left a comment
There was a problem hiding this comment.
LGTM, thanks for the contribution!
Follows the same pattern as commit 984398c which fixed manifest.name and manifest.homePage linting to use dynamic org.
This allows custom pipelines with different organizations to pass the multiqc_config lint check without needing to add exceptions.
PR checklist
CHANGELOG.mdis updateddocsis updated