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

WIP: Make JSONPath optional #235

Closed
wants to merge 1 commit into from
Closed

WIP: Make JSONPath optional #235

wants to merge 1 commit into from

Conversation

mmccool
Copy link
Contributor

@mmccool mmccool commented Nov 22, 2021

Should resolve a number of issues, see summary here: #234

  • mark JSONPath section as informative
  • edit intro to search section to say it is optional

TODO:

  • reword/remove RFC2119 assertions in body of JSONPath section
  • Need to update descriptions in TDD TD/TM

Preview | Diff

@mmccool mmccool requested a review from farshidtz November 22, 2021 14:58
@mmccool mmccool closed this Nov 22, 2021
@mmccool
Copy link
Contributor Author

mmccool commented Nov 22, 2021

Closed without merging, since @farshidtz 's PR #237 is cleaner, but I will review the result and see if any of my suggested changes should be added.

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