-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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 logger_factory.py #2328
Update logger_factory.py #2328
Conversation
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.
Maybe the changes in this PR is better fit into #2329 ?
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2328 +/- ##
==========================================
+ Coverage 38.14% 46.75% +8.60%
==========================================
Files 78 81 +3
Lines 7865 8085 +220
Branches 1683 1863 +180
==========================================
+ Hits 3000 3780 +780
+ Misses 4615 3983 -632
- Partials 250 322 +72
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Why are these changes needed?
Adding Cosmos DB for logging
Related issue number
N/A
Checks