-
Notifications
You must be signed in to change notification settings - Fork 438
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
[DNM] test adding slack notification for test summaries #10211
base: main
Are you sure you want to change the base?
Conversation
/skip-ci testing gha |
Visit the preview URL for this PR (updated for commit 7a32f65): https://gloo-edge--pr10211-jbohanon-test-summar-plmyfbcz.web.app (expires Thu, 24 Oct 2024 19:35:55 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 77c2b86e287749579b7ff9cadb81e099042ef677 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love the output, can we add the job / test name or something?
https://solo-io-corp.slack.com/archives/C0314KESVNV/p1729194404389549
@@ -37,6 +40,7 @@ runs: | |||
shell: bash | |||
run: make go-test | |||
- name: Summarize tests | |||
if: ${{ always() }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh my summarize wasnt always before? woof
Description
API changes
Code changes
CI changes
Docs changes
Context
Interesting decisions
Testing steps
Notes for reviewers
Checklist: