Skip to content

Conversation

taimoorzaeem
Copy link
Contributor

I think these are sufficient for my test coverage for now. I might add more in the future if need be.

Related #72.

Copy link
Collaborator

@gregsdennis gregsdennis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These pass for me. I'm not sure what the value of them is, but they're valid per the spec, so I suppose they should be included.

@gregsdennis gregsdennis requested a review from f3ath January 15, 2025 09:59
@gregsdennis
Copy link
Collaborator

@f3ath please feel free to merge once you've had a look.

Copy link
Collaborator

@f3ath f3ath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM and works on my end.

@f3ath f3ath merged commit 9d5e1b3 into jsonpath-standard:main Jan 15, 2025
2 checks passed
@taimoorzaeem taimoorzaeem deleted the filter/root-query branch January 15, 2025 17:10
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.

3 participants