-
Notifications
You must be signed in to change notification settings - Fork 25
[DX-2283] Adds branch-out uploading and flaky test protection #1823
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
base: main
Are you sure you want to change the base?
Conversation
|
👋 kalverra, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
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.
Pull request overview
This PR integrates the branch-out tool into CI workflows to detect and skip flaky tests while uploading test results to Trunk for analysis.
Changes:
- Replaces direct
go testcommands with thebranch-out-uploadGitHub action - Updates artifact naming from "go-test-results" to "go-test-coverage" to better reflect content
- Removes unnecessary blank lines for cleaner workflow formatting
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| .github/workflows/pkg.yml | Integrates branch-out action for unit tests and renames coverage artifact |
| .github/workflows/keystore.yml | Integrates branch-out action for unit tests and removes trailing blank line |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
✅ API Diff Results - No breaking changes |
Adds branch-out support to help guard CI against flaky tests, along with a bunch of other goodies.