-
Notifications
You must be signed in to change notification settings - Fork 3
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
Update BundleUpload status for observability #99
Conversation
65d855d
to
19c41c6
Compare
a21bdd1
to
a69301f
Compare
Summary: This change will help us give users visibility during onboarding. We can tell them how many uploads calls we've received and how many of them were successful. This will hopefully translate into less time users have to spend waiting to get value from flaky tests. Closes TRUNK-12756 Test Plan: TODO
✅ 75 passed ⋅ (learn more) |
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.
Very nice! Thank you so much 👏
@dfrankland mind giving this a quick re-review? Since your last approval - resolved some merge conflicts, fixed a bad enum value, and refactored the API call in |
Summary:
This change will help us give users visibility during onboarding.
We can tell them how many uploads calls we've received and how many of them were successful.
This will hopefully translate into less time users have to spend waiting to get value from flaky tests.
Closes TRUNK-12756
Can't be merged until the corresponding API changes (https://github.com/trunk-io/trunk/pull/16848, https://github.com/trunk-io/trunk/pull/16855) are in. The checks will continue to fail until then.