-
Notifications
You must be signed in to change notification settings - Fork 471
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
Add buildpulse #3051
Add buildpulse #3051
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3051 +/- ##
==========================================
- Coverage 43.68% 43.65% -0.04%
==========================================
Files 390 390
Lines 86681 86681
==========================================
- Hits 37868 37840 -28
- Misses 42796 42818 +22
- Partials 6017 6023 +6
Continue to review full report at Codecov.
|
@onetechnical @algobarb @algojack this is ready for review/merge |
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.
lgtm
Are we signing up for buildpulse @onetechnical ?https://github.com/marketplace/buildpulse/plan/MDIyOk1hcmtldHBsYWNlTGlzdGluZ1BsYW40NzM5#pricing-and-setup |
Yes. Chris has been POCing it to see if we can get better test result evaluation. |
This adds buildpulse integration using their orb.
Summary
This adds buildpulse integration by downloading their reporter for each platform we run tests on, and comparing it to known checksums.
Test Plan
CI changes only