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

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

Abnf cleanup #353

Closed cabo closed 1 year ago

cabo commented 1 year ago

Close #344 (Once completed)

goessner commented 1 year ago

Erroneously commented to already closed https://github.com/ietf-wg-jsonpath/draft-ietf-jsonpath-base/issues/344#issuecomment-1382692603 and https://github.com/ietf-wg-jsonpath/draft-ietf-jsonpath-base/issues/344#issuecomment-1382693954

cabo commented 1 year ago

Thanks. #344 is not yet closed; completing this #353 is meant to do that. The triple dot bug was already fixed here in 084c822. For me it doesn't matter where additional comments are made.

I wonder whether we should finish this round (fixing bugs) before doing a grand renaming. I also would like some feedback on the names proposed...

goessner commented 1 year ago

well, finishing this round before going on is fine.

cabo commented 1 year ago

I think if we agree this we should merge it. Next step could be the great renaming, possibly with some further testing of the grammar. In particular, we should throw collections of JSONPath queries we can find at it. If you have pointers, I could prepare that.

glyn commented 1 year ago

@cabo Two sources of queries: