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

make mod parser public #26

Closed
wants to merge 9 commits into from
Closed

Conversation

gkorland
Copy link
Contributor

ref #18 exposing the parsing results will allow us to use the ParseTokens to add new paths

ref #18 exposing the parsing results will allow us to use the ParseTokens to add new paths
@gkorland
Copy link
Contributor Author

gkorland commented Sep 1, 2019

@freestrings can you please merge this PR?
Based on this PR we can write a NodeVisitor and work around issue #18

@gkorland
Copy link
Contributor Author

gkorland commented Oct 7, 2019

@freestrings

@freestrings
Copy link
Owner

@gkorland
I have a lot of work these days, so I checked "mail" late.

The parser was developed for module internal use and plans to support joins and functions. this module is likely to change. there are no plans to make it public yet.

@gkorland
Copy link
Contributor Author

@freestrings any other option you might expose this functionality?

@gkorland
Copy link
Contributor Author

gkorland commented Feb 6, 2020

@freestrings that is only way I found to verify a path is static and can return at most one path

@ctron ctron mentioned this pull request Dec 9, 2020
@ctron
Copy link
Contributor

ctron commented Dec 9, 2020

@freestrings @gkorland

I would be interested in the same feature: see #61 #62

As the PR is open for over a year now, with no progress and feedback, I am wondering if this crate is still maintained. I think it is ok, if it is not. But that would be good to know, so I might consider forking and re-publishing the crate myself.

@freestrings
Copy link
Owner

Fixed issue at https://github.com/freestrings/jsonpath/tree/feature/refactoring and will be merged soon.

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