Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update ci-testing.yml #11642

Merged
merged 1 commit into from
Jun 1, 2023
Merged

Update ci-testing.yml #11642

merged 1 commit into from
Jun 1, 2023

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Jun 1, 2023

πŸ€– Generated by Copilot at 75fe068

Summary

πŸ”„βŒπŸš¨

Improved Slack notification for CI testing workflow. Added cancelled status check for Benchmarks and Tests jobs in .github/workflows/ci-testing.yml.

Benchmarks and Tests
Cancelled or failed, they send
Slack alert in spring

Walkthrough

  • Update Slack notification condition to include cancelled status of Benchmarks and Tests jobs (link). This ensures that any potential issues that might cause the workflow to be cancelled prematurely are reported to the Slack channel.

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Expanded CI notification conditions to include cancelled workflows in the Ultralytics YOLOv5 repository.

πŸ“Š Key Changes

  • Modified the Continuous Integration (CI) testing workflow to trigger notifications not only on failure but also when jobs are cancelled.

🎯 Purpose & Impact

  • Purpose: Ensure the development team is notified if a CI job is cancelled, in addition to the existing notifications for job failures, to keep track of the CI process more effectively.
  • Impact: Expanding the notification triggers improves workflow monitoring and can help address issues more promptly, maintaining the project's stability and quality. It keeps the team informed of all critical CI workflow statuses, helping to prevent unnoticed disruptions in the development process. πŸš€

Signed-off-by: Glenn Jocher <[email protected]>
@glenn-jocher glenn-jocher merged commit 5f11555 into master Jun 1, 2023
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch June 1, 2023 11:06
NagatoYuki0943 added a commit to NagatoYuki0943/yolov5-ultralytics that referenced this pull request Jun 2, 2023
bandakopi pushed a commit to irajcode/yolov5 that referenced this pull request Jul 20, 2023
pleb631 pushed a commit to pleb631/yolov5 that referenced this pull request Jan 6, 2024
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.

1 participant