+
Skip to content

Conversation

matthdsm
Copy link
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
Copy link
Contributor

mashehu commented Sep 29, 2025

@nf-core-bot changelog

@mashehu
Copy link
Contributor

mashehu commented Sep 29, 2025

@nf-core-bot changelog

Copy link

codecov bot commented Sep 29, 2025

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
Copy link
Contributor

mashehu commented Sep 29, 2025

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

Copy link
Member

@mirpedrol mirpedrol left a comment

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

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载