-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
chore(integrations): SourceCodeSearchEndpoint metrics #80956
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is ✅ All tests successful. No failed tests found.
Additional details and impacted files@@ Coverage Diff @@
## master #80956 +/- ##
==========================================
+ Coverage 80.32% 80.33% +0.01%
==========================================
Files 7215 7215
Lines 319443 319409 -34
Branches 20773 20773
==========================================
+ Hits 256583 256604 +21
+ Misses 62466 62411 -55
Partials 394 394 |
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.
where we record failure vs halt all lgtm, just a note about adding a reason for recording halts.
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.
Nice work! We just need a bit more context on the halt cases and we should be good to go.
assert halt1.args[0] == EventLifecycleOutcome.SUCCESS | ||
assert halt2.args[0] == EventLifecycleOutcome.SUCCESS |
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.
Super minor naming Nit: these are success metrics, not halts
465ca5c
to
d0b79e0
Compare
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.
Add metrics to
handle_search_issues
,handle_search_repositories
, andget
.