Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix failures from latest compliance test suite #90

Closed
hiltontj opened this issue May 9, 2024 · 0 comments · Fixed by #92
Closed

Fix failures from latest compliance test suite #90

hiltontj opened this issue May 9, 2024 · 0 comments · Fixed by #92
Labels
bug Something isn't working compliance JSONPath Standard Compliance

Comments

@hiltontj
Copy link
Owner

hiltontj commented May 9, 2024

The tests added in jsonpath-standard/jsonpath-compliance-test-suite#35 fail on serde_json_path. This may be fixed by using the CRLF mode in the rust regex crate (see rust-lang/regex#244 (comment))
@hiltontj hiltontj added bug Something isn't working compliance JSONPath Standard Compliance labels May 9, 2024
@hiltontj hiltontj mentioned this issue May 12, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working compliance JSONPath Standard Compliance
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: use CRLF mode in regexes for match and search hiltontj/serde_json_path
1 participant
@hiltontj