-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
fix: remove unused TestContext #1013
Conversation
The function `WithTestContext` is never called in the project, and there is just one usage of `TestContext`, inside one of our loggers
Codecov Report
@@ Coverage Diff @@
## master #1013 +/- ##
==========================================
+ Coverage 57.50% 58.90% +1.39%
==========================================
Files 32 33 +1
Lines 4594 4645 +51
==========================================
+ Hits 2642 2736 +94
+ Misses 1729 1681 -48
- Partials 223 228 +5
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
A bit of context: We've introduced the |
@robertkowalski this pull request has failed checks 🛠 |
@robertkowalski this pull request has failed checks 🛠 |
The function
WithTestContext
is never called in the project, andthere is just one usage of
TestContext
, inside one of our loggers