Skip to content

ci: send a notification of scheduled failing tests - #1323

Merged
zimeg merged 17 commits into
mainfrom
zimeg-ci-notifications
Jun 27, 2025
Merged

ci: send a notification of scheduled failing tests#1323
zimeg merged 17 commits into
mainfrom
zimeg-ci-notifications

Conversation

@zimeg

Copy link
Copy Markdown
Member

Summary

This PR sends a notification of scheduled failing tests using the new "regression failures" workflow!

Testing

On schedule and according to #1322 🫡

Notes

I plan to test a few combinations in this PR to make sure notifications are sent 🚀 ✨

Category

  • Others

Requirements

Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

  • I've read and understood the Contributing Guidelines and have done my best effort to follow them.
  • I've read and agree to the Code of Conduct.
  • I've run ./scripts/install_all_and_run_tests.sh after making the changes.

@zimegzimeg self-assigned this Jun 27, 2025
@zimegzimeg added github_actions Pull requests that update GitHub Actions code tests labels Jun 27, 2025
@codecov

codecovBot commented Jun 27, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.96%. Comparing base (079552d) to head (afad2df).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@ Coverage Diff @@## main #1323 +/- ##
==========================================
+ Coverage 90.94% 90.96% +0.01% 
==========================================
Files 222 222 Lines 7501 7501 ==========================================
+ Hits 6822 6823 +1 + Misses 679 678 -1 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Comment thread.github/workflows/tests.yml Outdated
name: Regression notifications
runs-on: ubuntu-latest
needs: build
if: failure() && github.ref == 'refs/heads/zimeg-ci-notifications' && github.event_name != 'workflow_dispatch'

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

📣 workflow_dispatch will not send a notification!

🔗 https://github.com/slackapi/bolt-python/actions/runs/15935708998

@WilliamBergaminWilliamBergamin 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.

Sweet 💯 nice work!!

@zimeg

Copy link
Copy Markdown
MemberAuthor

@WilliamBergamin Likewise, thank you for helping in the iterations above 🙏 ✨

I will merge this now but I hope we don't get notifications for this 😉

@zimeg
zimeg merged commit d3ca055 into mainJun 27, 2025
@zimeg
zimeg deleted the zimeg-ci-notifications branch June 27, 2025 21:10
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actionsPull requests that update GitHub Actions codetests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@zimeg@WilliamBergamin