Skip to content

SONARJAVA-6213 Delete obsolete Slack notification workflow#5535

Merged
asya-vorobeva merged 1 commit intomasterfrom
tt/delete-slack-notification
Apr 3, 2026
Merged

SONARJAVA-6213 Delete obsolete Slack notification workflow#5535
asya-vorobeva merged 1 commit intomasterfrom
tt/delete-slack-notification

Conversation

@tomasz-tylenda-sonarsource
Copy link
Copy Markdown
Contributor

@tomasz-tylenda-sonarsource tomasz-tylenda-sonarsource commented Mar 26, 2026

No description provided.

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title Delete obsolete Slack notification workflow SONARJAVA-6213 Delete obsolete Slack notification workflow Mar 26, 2026
@sonar-review-alpha
Copy link
Copy Markdown
Contributor

sonar-review-alpha bot commented Mar 26, 2026 •
edited
Loading

Summary

Deletes the slack_notify.yml GitHub Actions workflow that was sending notifications to a squad-jvm-notifs Slack channel whenever check suites completed on main, master, or specific branch patterns (dogfood-, branch-). This workflow is no longer needed.

What reviewers should know

This is a straightforward deletion of a single workflow file. Reviewers should verify that: 1) this Slack notification mechanism is genuinely obsolete (e.g., notifications moved to a different tool or channel), and 2) no team processes depend on these notifications for monitoring builds/tests. Check the PR discussion or linked issue for context on why this workflow is being removed.

  • Generate Walkthrough
  • Generate Diagram

🗣️ Give feedback

@hashicorp-vault-sonar-prod
Copy link
Copy Markdown
Contributor

hashicorp-vault-sonar-prod bot commented Mar 26, 2026 •
edited by atlassian bot
Loading

SONARJAVA-6213

@sonarqube-next
Copy link
Copy Markdown

sonarqube-next bot commented Mar 26, 2026

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

sonar-review-alpha[bot]
sonar-review-alpha bot reviewed Mar 26, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@sonar-review-alpha sonar-review-alpha bot left a comment

Choose a reason for hiding this comment

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

LGTM! ✅

🗣️ Give feedback

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026

This PR is stale because it has been open 7 days with no activity. If there is no activity in the next 7 days it will be closed automatically

@github-actions github-actions bot added the stale label Apr 3, 2026
@asya-vorobeva asya-vorobeva merged commit 79aacab into master Apr 3, 2026
19 checks passed
@asya-vorobeva asya-vorobeva deleted the tt/delete-slack-notification branch April 3, 2026 10:03
leonardo-pilastri-sonarsource pushed a commit that referenced this pull request Apr 8, 2026
@tomasz-tylenda-sonarsource @leonardo-pilastri-sonarsource
SONARJAVA-6213 Delete obsolete Slack notification workflow (#5535)
457abda
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sonar-review-alpha sonar-review-alpha[bot] sonar-review-alpha[bot] left review comments

@asya-vorobeva asya-vorobeva asya-vorobeva approved these changes

Assignees

No one assigned

Labels

stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tomasz-tylenda-sonarsource @asya-vorobeva