-
Notifications
You must be signed in to change notification settings - Fork 51
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
feat: Add JSON scalar #2254
feat: Add JSON scalar #2254
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.
LGTM. Thanks for adding this type 👍
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #2254 +/- ##
===========================================
+ Coverage 74.07% 74.09% +0.02%
===========================================
Files 256 256
Lines 25731 25767 +36
===========================================
+ Hits 19058 19090 +32
- Misses 5356 5360 +4
Partials 1317 1317
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 7 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Co-authored-by: Shahzad Lone <[email protected]>
## Relevant issue(s) Resolves sourcenetwork#2243 ## Description This PR adds a JSON scalar type to the schema system ## Tasks - [x] I made sure the code is well commented, particularly hard-to-understand areas. - [x] I made sure the repository-held documentation is changed accordingly. - [x] I made sure the pull request title adheres to the conventional commit style (the subset used in the project can be found in [tools/configs/chglog/config.yml](tools/configs/chglog/config.yml)). - [x] I made sure to discuss its limitations such as threats to validity, vulnerability to mistake and misuse, robustness to invalidation of assumptions, resource requirements, ... ## How has this been tested? make test Specify the platform(s) on which this was tested: - MacOS --------- Co-authored-by: Shahzad Lone <[email protected]>
bug bash issue #2370 |
Relevant issue(s)
Resolves #2243
Description
This PR adds a JSON scalar type to the schema system
Tasks
How has this been tested?
make test
Specify the platform(s) on which this was tested: