ietf-wg-jsonpath / draft-ietf-jsonpath-base

Development of a JSONPath internet draft
https://ietf-wg-jsonpath.github.io/draft-ietf-jsonpath-base/
Other
59 stars 20 forks source link

Clarify JSON values early in the spec #300

Closed glyn closed 1 year ago

glyn commented 1 year ago

Fixes https://github.com/ietf-wg-jsonpath/draft-ietf-jsonpath-base/issues/299

glyn commented 1 year ago

@cabo I take your point about this PR not being easy to review. I will think again.

cabo commented 1 year ago

(Let me just add that being explicit about these things is a significant improvement, but the explanations should not assume that the reader has a specific misunderstanding and try to talk them out of it -- that creates new misunderstandings.)

glyn commented 1 year ago

(Let me just add that being explicit about these things is a significant improvement, but the explanations should not assume that the reader has a specific misunderstanding and try to talk them out of it -- that creates new misunderstandings.)

Very helpful thanks. Let's address this change with a few tiny PRs.