Skip to content

SCAN4NET-1878 Direct Slack notifications to .NET squad channels - #3540

Merged
pavel-mikula-sonarsource merged 3 commits into
masterfrom
mary/slack-squad-dotnet-only
Nov 13, 2026
Merged

pavel-mikula-sonarsource merged 3 commits into
masterfrom
mary/slack-squad-dotnet-only

Conversation

@mary-georgiou

Copy link
Copy Markdown
Contributor

Remove squad-corelang-notifs from the Slack report channels; keep only squad-dotnet.

🤖 Generated with Claude Code

Remove squad-corelang-notifs from the CI report channels.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod Bot changed the title Notify only squad-dotnet Slack channel SCAN4NET-1878 Notify only squad-dotnet Slack channel Nov 10, 2026
@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

SCAN4NET-1878

@datadog-sonarsource

datadog-sonarsource Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

Pipelines

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 392fe68 | Docs | Give us feedback!

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Comment thread .github/workflows/ci.yml Resolved
@mary-georgiou mary-georgiou changed the title SCAN4NET-1878 Notify only squad-dotnet Slack channel SCAN4NET-1878 Notify only squad-dotnet-notifs Slack channel Nov 11, 2026
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@mary-georgiou mary-georgiou changed the title SCAN4NET-1878 Notify only squad-dotnet-notifs Slack channel SCAN4NET-1878 Route Slack notifications: CI failures to squad-dotnet-notifs, releases to squad-dotnet Nov 11, 2026
@mary-georgiou mary-georgiou changed the title SCAN4NET-1878 Route Slack notifications: CI failures to squad-dotnet-notifs, releases to squad-dotnet SCAN4NET-1878 Direct Slack notifications to .NET squad channels Nov 11, 2026
@sonarqubecloud

Copy link
Copy Markdown

@gitar-bot

gitar-bot Bot commented Nov 11, 2026

Copy link
Copy Markdown
Code Review ✅ Approved 🟡 Medium risk 1 closed / 1 findings

Workflow changes route CI, release, and lock-branch Slack messages to .NET channels, potentially misdirecting operational notifications.

Restricts Slack release notifications to the squad-dotnet-notifs channel, removing squad-corelang-notifs from the report channels. The mismatch between the PR description's squad-dotnet name and the code's squad-dotnet-notifs channel was resolved. No open issues found.

✅ 1 closed
✅ Quality: PR title/description name squad-dotnet, but code now uses squad-dotnet-notifs

📄 .github/workflows/ci.yml:432
The PR title ("Notify only squad-dotnet Slack channel") and description ("keep only squad-dotnet") say CI failure reports go to squad-dotnet. After the latest commit, ci.yml sends them to squad-dotnet-notifs instead. Anyone reading the PR or the squash-merge message will look in the wrong channel. Update the title and description to say squad-dotnet-notifs, or switch the channel back if squad-dotnet was the intended target.

Verification Rules 3/3 met

Functional validation No results

Rules 3 passed

✅ Approved GitHub Actions runners
✅ Internal artifact registries
✅ Repository license compliance

Auto-approval Not enabled

Set up

For more verification details, comment gitar display:verbose.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing compact verification results.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@pavel-mikula-sonarsource pavel-mikula-sonarsource left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@pavel-mikula-sonarsource
pavel-mikula-sonarsource merged commit 2e1e6e5 into master Nov 13, 2026
30 checks passed
@pavel-mikula-sonarsource
pavel-mikula-sonarsource deleted the mary/slack-squad-dotnet-only branch November 13, 2026 07:17
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.

2 participants