Skip to content

Template: Update nextflow.config to use environment variable for hook_url#3756

Merged
mashehu merged 6 commits into
devfrom
matthdsm-patch-1
Oct 1, 2025
Merged

Template: Update nextflow.config to use environment variable for hook_url#3756
mashehu merged 6 commits into
devfrom
matthdsm-patch-1

Conversation

@matthdsm

Copy link
Copy Markdown
Contributor

So the hook_url can be set in the env for every pipeline

PR checklist

  • This comment contains a description of changes (with reason)
  • CHANGELOG.md is updated
  • If you've fixed a bug or added code that should be tested, add tests!
  • Documentation in docs is updated

@mashehu mashehu changed the title Update hook_url to use environment variable Template: Update nextflow.config to use environment variable for hook_url Sep 29, 2025
@mashehu

mashehu commented Sep 29, 2025

Copy link
Copy Markdown
Contributor

@nf-core-bot changelog

@mashehu

mashehu commented Sep 29, 2025

Copy link
Copy Markdown
Contributor

@nf-core-bot changelog

@codecov

codecov Bot commented Sep 29, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.33%. Comparing base (4475032) to head (bf57750).
⚠️ Report is 7 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 changed the title Template: Update nextflow.config to use environment variable for hook_url Template: Update nextflow.config to use environment variable for hook_url Sep 29, 2025
@mashehu

mashehu commented Sep 29, 2025

Copy link
Copy Markdown
Contributor

@nf-core-bot changelog Template: Update nextflow.config to use environment variable for hook_url

@mirpedrol mirpedrol left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@mashehu mashehu enabled auto-merge October 1, 2025 16:11
@mashehu mashehu merged commit 01ac55e into dev Oct 1, 2025
115 checks passed
@mashehu mashehu deleted the matthdsm-patch-1 branch October 1, 2025 16:18
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