-
Notifications
You must be signed in to change notification settings - Fork 28
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
Revise a need of JsonPathFinder and groom the API in general. #59
Comments
@besok 3 potential API changes would also be:
|
I can have a look into these points after #58 got merged. Not a huge fan of gigantic refactoring PRs, rather do it in smaller steps. What do you think ? |
@besok any comment at this ? maybe any way to make communication easier than via github ? Like discord ? |
Hey! Sorry, I am on vacation and very slowly respond. Yeah, sure. You can take it and split it into a smaller prs. I will finish the review of the current pr this week. |
JsonPathFinder seems to be redundant. Probably, better to revise the API a remove it
The text was updated successfully, but these errors were encountered: