Skip to content

Jsonpath #2272

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

Merged
merged 5 commits into from
Oct 8, 2024
Merged

Jsonpath #2272

merged 5 commits into from
Oct 8, 2024

Conversation

lemire
Copy link
Member

@lemire lemire commented Oct 5, 2024

PR by @yangzhg at #2266
...

  • Updated json_path_to_pointer_conversion to support JSONPath starting with the '$' prefix, while maintaining compatibility with the existing implementation.
  • Moved json_path_to_pointer_conversion to a separate header file for better modularity and to support JSONPath queries in dom mode.
  • Implemented at_path functionality in dom mode to enable querying JSON using JSONPath.
  • Added unit tests to validate the new JSONPath support in dom mode and ensure compatibility with both standard and existing JSONPath formats.

This new PR adds two minor fixes (documentation + header inclusion).

yangzhg and others added 5 commits October 4, 2024 22:49
…ath (#2266)

- Updated `json_path_to_pointer_conversion` to support JSONPath starting with the '$' prefix, while maintaining compatibility with the existing implementation.
- Moved `json_path_to_pointer_conversion` to a separate header file for better modularity and to support JSONPath queries in `dom` mode.
- Implemented `at_path` functionality in `dom` mode to enable querying JSON using JSONPath.
- Added unit tests to validate the new JSONPath support in `dom` mode and ensure compatibility with both standard and existing JSONPath formats.
@lemire lemire merged commit c066b54 into master Oct 8, 2024
131 checks passed
@lemire lemire deleted the jsonpath branch October 8, 2024 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants