-
Notifications
You must be signed in to change notification settings - Fork 29
Explicit definition of how to recursively merge objects #216
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
Conversation
And an extended Traits example
5384f42 to
915d5c0
Compare
mikekistler
left a comment
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. 👍
Let's fix the target while we're at it.
Co-authored-by: Mike Kistler <[email protected]>
mikekistler
left a comment
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! 👍
| version: 1.0.0 | ||
| actions: | ||
| - target: $.paths.*.get[[email protected].paged] | ||
| - target: $.paths.*.get[?(@.x-oai-traits[?(@ == 'paged')])] |
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.
leaving a note to check at a later time whether it's fully compliant with RFC 9535 since both JsonPath.net and https://jsonpath.com/ complain about this one.
lornajane
left a comment
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.
Thanks!
And an extended Traits example.
Fixes #63.