-
Notifications
You must be signed in to change notification settings - Fork 81
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
Peter/fix orjson #192
Peter/fix orjson #192
Conversation
) | ||
|
||
|
||
def default(obj): |
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.
please add typing
requirements/_requirements.txt
Outdated
@@ -18,3 +18,4 @@ requests-toolbelt>=1.0.0 | |||
wheel>=0.38.1 | |||
setuptools>=65.5.1 | |||
pytest-asyncio<=0.21.1 | |||
orjson |
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 a better placement would be requirements.http.txt?
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.
Agreed! And can we pin orjson?
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.
Looking good! Left one comment in addition to Paweł's comments.
requirements/_requirements.txt
Outdated
@@ -18,3 +18,4 @@ requests-toolbelt>=1.0.0 | |||
wheel>=0.38.1 | |||
setuptools>=65.5.1 | |||
pytest-asyncio<=0.21.1 | |||
orjson |
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.
Agreed! And can we pin orjson?
Description
Reintroduce orjson with fixes for visualizations in responses
Add visualization tests to regression testing
Type of change
Please delete options that are not relevant.
How has this change been tested, please provide a testcase or example of how you tested the change?
Locally
Any specific deployment considerations
Non
Docs