-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
JSON encoder change via app #2055
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2055 +/- ##
=============================================
+ Coverage 91.995% 92.007% +0.012%
=============================================
Files 36 36
Lines 3298 3303 +5
Branches 562 564 +2
=============================================
+ Hits 3034 3039 +5
Misses 179 179
Partials 85 85
Continue to review full report at Codecov.
|
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.
Looks good to me. Thanks for taking care of some of the import sorting issues.
Thanks |
Resolves #2054
This one has come up a few times in the past. The current method for setting this at the default level by only looking at the environment is not ideal.
Adds the following API to set the encoder globally.