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.

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

Update CTS and fix #23

Merged
merged 1 commit into from
Aug 5, 2024
Merged

Update CTS and fix #23

merged 1 commit into from
Aug 5, 2024

Conversation

jg-rp
Copy link
Owner

@jg-rp jg-rp commented Aug 4, 2024

This PR updates the CTS and fixes:

  • The default minimum integer boundary for JSONPath indexes and slice steps. We were off by one.
  • Parsing of JSONPath integer literals with an exponent and an upper case 'e'. We now allow 'e' to be upper case.
  • Handling of trailing commas in JSONPath bracketed segments. We now raise a syntax error.
  • Handling of invalid JSONPath integer and float literals with extra minus signs, leading zeros or too many zeros. We now raise a syntax error in such cases.

See jsonpath-standard/jsonpath-compliance-test-suite#89.

@jg-rp jg-rp merged commit e60678b into main Aug 5, 2024
12 checks passed
@jg-rp jg-rp deleted the cts-fixes branch August 5, 2024 06:40
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.

1 participant