-
Notifications
You must be signed in to change notification settings - Fork 494
chore (internal/civisibility): remove the unused impacted test backend call #3473
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 (internal/civisibility): remove the unused impacted test backend call #3473
Conversation
BenchmarksBenchmark execution time: 2025-05-29 12:24:31 Comparing candidate commit 04d7438 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 21 metrics, 0 unstable metrics. |
9e7a0cd to
b32841e
Compare
Datadog ReportBranch report: ✅ 0 Failed, 4523 Passed, 64 Skipped, 3m 32.66s Total Time |
anmarchenko
left a comment
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.
this looks already good, the changes will be not so big to work with remote branches only
What does this PR do?
This PR removes the backend call to the impacted tests endpoint.
Motivation
We decided that all the calculations are going to be made on the tracer, so we don't need the backend fallback anymore.
Documented here: https://docs.google.com/document/d/13EBqv-8XKGAmDR4J10Lwu1UQYXGfYUwjmOBDQaKKqbw
Reviewer's Checklist
golangci-lint runlocally.Unsure? Have a question? Request a review!