FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

ci: send a notification of scheduled failing tests by zimeg · Pull Request #1323 · slackapi/bolt-python · GitHub

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 commented Jun 27, 2025

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.

zimeg requested a review from WilliamBergamin June 27, 2025 20:15
zimeg self-assigned this Jun 27, 2025
zimeg added github_actions Pull requests that update GitHub Actions code tests labels Jun 27, 2025

codecov Bot commented Jun 27, 2025
edited
Loading

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
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

📣 workflow_dispatch will not send a notification!

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

WilliamBergamin 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Sweet 💯 nice work!!

zimeg commented Jun 27, 2025

Copy link
Copy Markdown
Member Author

@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 merged commit d3ca055 into main Jun 27, 2025
zimeg deleted the zimeg-ci-notifications branch June 27, 2025 21:10
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL