Conversation
Codecov Report
@@ Coverage Diff @@
## master #101 +/- ##
==========================================
+ Coverage 92.71% 92.73% +0.01%
==========================================
Files 21 21
Lines 1139 1142 +3
Branches 140 141 +1
==========================================
+ Hits 1056 1059 +3
Misses 55 55
Partials 28 28
Continue to review full report at Codecov.
|
@@ -49,6 +51,7 @@ def __init__( | |||
self, | |||
connection_string: str = None, | |||
instrumentation_key: str = None, | |||
proxies: typing.Dict[str, str] = None, |
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 setting the default as {} instead of None to avoid check afterwards
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.
Mutable defaults are a common pitfall in Python. https://docs.python-guide.org/writing/gotchas/#mutable-default-arguments
Add the ability to specify proxies that the request to Azure Monitor backend will pass through.
Users will pass in a string representing a dict format.
See requests for more details.