Skip to content

Replace webhook notifications with nf-slack plugin#4051

Merged
FriederikeHanssen merged 8 commits intonf-core:devfrom
FriederikeHanssen:nf-slack-bot
Feb 27, 2026
Merged

Replace webhook notifications with nf-slack plugin#4051
FriederikeHanssen merged 8 commits intonf-core:devfrom
FriederikeHanssen:nf-slack-bot

Conversation

@FriederikeHanssen
Copy link
Copy Markdown
Contributor

@FriederikeHanssen FriederikeHanssen commented Feb 18, 2026

Summary

  • Replace the old webhook-based Slack/Teams notification system (hook_url, slackreport.json, adaptivecard.json, imNotification()) with the nf-slack plugin
  • The plugin is declared entirely within the CI workflow's nextflow_config block, keeping local runs completely unaffected
  • Remove adaptivecard and slackreport template features

Test plan

  • Verify template renders correctly with nf-core pipelines create
  • Confirm no remaining references to hook_url, slackreport, adaptivecard, or imNotification
  • Test cloud workflow with nf-slack bot token on a pipeline

🤖 Generated with Claude Code

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 18, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.39%. Comparing base (1dddccf) to head (12d6fae).
⚠️ Report is 9 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.

@FriederikeHanssen @claude
Replace webhook notifications with nf-slack plugin in pipeline template
b0359d5 
Replace the old webhook-based Slack/Teams notification system (hook_url,
slackreport.json, adaptivecard.json, imNotification function) with the
nf-slack plugin. The plugin is declared entirely within the CI workflow's
nextflow_config block, keeping local runs completely unaffected.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@FriederikeHanssen FriederikeHanssen mentioned this pull request Feb 18, 2026
Merged
2 tasks
mashehu
mashehu reviewed Feb 18, 2026
View reviewed changes
Comment thread nf_core/pipeline-template/subworkflows/nf-core/utils_nfcore_pipeline/main.nf
mashehu
mashehu reviewed Feb 18, 2026
View reviewed changes
Comment thread nf_core/pipelines/create/template_features.yml
@FriederikeHanssen @claude
Revert imNotification removal from nf-core subworkflow
d984c91 
This change needs to happen upstream in nf-core/modules, not here.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@FriederikeHanssen FriederikeHanssen marked this pull request as draft February 19, 2026 15:28
@FriederikeHanssen FriederikeHanssen marked this pull request as ready for review February 20, 2026 10:29
@mashehu
Copy link
Copy Markdown
Contributor

mashehu commented Feb 20, 2026

this code is not just for slack notficiations, it also includes MS teams notifications. I know that @nvnieuwk is working on it, but until we have that, I don't think we can remove this code completely, but just the slack part.

@FriederikeHanssen
Copy link
Copy Markdown
Contributor Author

FriederikeHanssen commented Feb 20, 2026

It has a release here https://registry.nextflow.io/plugins/nf-teams@0.1.0 waiting for Nicolas to confirm though

@nvnieuwk
Copy link
Copy Markdown
Contributor

nvnieuwk commented Feb 23, 2026

Yes the plugin is already feature compatible with what is currently in the template, with some nice updates and extra features added to it. There currently is no workflow agent support yet as I just need to find the time to figure that out properly (but not a problem since the template also didn't support that)

@FriederikeHanssen
Copy link
Copy Markdown
Contributor Author

FriederikeHanssen commented Feb 23, 2026

@mashehu is this sufficient for you?

how do I update the snapshots the test is complaining about?

@mirpedrol
Copy link
Copy Markdown
Member

mirpedrol commented Feb 24, 2026

@nf-core-bot update snapshots

@mirpedrol
Copy link
Copy Markdown
Member

mirpedrol commented Feb 24, 2026

@nf-core-bot update textual snapshots

mashehu
mashehu approved these changes Feb 24, 2026
View reviewed changes
Comment thread CHANGELOG.md Outdated
@mashehu
Copy link
Copy Markdown
Contributor

mashehu commented Feb 24, 2026

sounds good then.

FriederikeHanssen and others added 2 commits February 27, 2026 18:54
@FriederikeHanssen @mashehu
Update CHANGELOG.md
6ff069e 
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
@FriederikeHanssen
Merge branch 'dev' into nf-slack-bot
8e092ca
@FriederikeHanssen
Copy link
Copy Markdown
Contributor Author

FriederikeHanssen commented Feb 27, 2026

@nf-core-bot fix linting

@FriederikeHanssen
Copy link
Copy Markdown
Contributor Author

FriederikeHanssen commented Feb 27, 2026

@nf-core-bot update textual snapshots

@FriederikeHanssen FriederikeHanssen merged commit a800f44 into nf-core:dev Feb 27, 2026
112 checks passed
@FriederikeHanssen FriederikeHanssen deleted the nf-slack-bot branch February 27, 2026 18:19
@maxulysse maxulysse mentioned this pull request Mar 2, 2026
Merged
11 tasks
@muffato muffato mentioned this pull request Mar 6, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mashehu mashehu mashehu approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@FriederikeHanssen @mashehu @nvnieuwk @mirpedrol @nf-core-bot