Simplify Slack notification messages#324
Conversation
|
👋 Hello @glenn-jocher, thank you for submitting a
For more guidance, please refer to our Contributing Guide. Don't hesitate to leave a comment if you have any questions. Thank you for contributing to Ultralytics! 🚀 |
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review
Made with ❤️ by Ultralytics Actions
PR looks clean. The Slack payload simplification is straightforward, scoped to the changed line, and I don't see any clear bug or maintainability issue introduced by this diff.
|
Merged — fantastic improvement, @glenn-jocher! 🚀 As Blaise Pascal said, “I would have written a shorter letter, but I did not have the time.” This update captures that idea perfectly: a cleaner, more focused Slack alert that reaches the right team faster and reduces noise for everyone else. Thanks for sharpening the workflow experience with a small change that delivers clear operational value. |
Replaces the verbose multi-line Slack notifications with a single dense line — bold workflow name, status emoji,
${{ github.repository }}, and a Run backlink — matching the format used inultralytics/portal.<!channel>ping down to@marketing-teamto cut company-wide notification noise.🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
🔔 This PR updates the Slack alert message in the domain-check GitHub Actions workflow to make failure notifications shorter, clearer, and directed to a specific team group.
📊 Key Changes
<!channel>alert to a targeted Slack user group mention:<!subteam^S07ABRC4MLN>..github/workflows/check_domains.yml.🎯 Purpose & Impact