Open Skitionek opened 4 years ago
We just pushed yet another fork this one utilizing v8-sandbox as an alternative / safer expression evaluation backend. It can be accessed here
@movitto Nice! I assume that one is Node.js only though which doesn't serve my particular purpose. I've also added a fork on the pile at https://github.com/astronautlabs/jsonpath
This one is converted to Typescript, tests converted to Razmin, uses CircleCI to match the rest of our stack, and we'll be maintaining it for the foreseeable future as it is core to one of our upcoming products over at Astronaut Labs. We welcome PRs!
I think there's a bigger problem for this whole community: jison is very very unmaintained and there is this alarming issue there: https://github.com/zaach/jison/issues/390
It's not clear if the issue identified there affects the generated output of jison or the library itself...
@rezonant glad to hear. Yes our fork is currently nodejs only, perhaps we'll investigate if it can be exposed to a browser environment in the future.
Appreciate the heads up WRT JISON. There have been some suggestions about replacing it (see #127) though haven't seen any manifestations of that yet.
I did take a look at jison-gho, got a ways into switching my fork to use it but ended up rolling it back. I don't recall the exact reasoning. Plus jison-gho isn't heavily maintained either...
If the library is stable activity isn't a big issue as changes are rarely needed. But yes it does seem like jison-gho has various important issues / PRs open that have yet to be addressed
I switched to https://www.npmjs.com/package/jsonpath-plus
Although it's clear potential this repository seems to not be properly maintain in last 3 years. Among these 125 forks, does any of them is maintained?
To the author: Please do open this repository for collaboration, it might be good idea to add current "forkers" to collaborator team. Going step further by github actions you can restrict push to master to ones passing tests, and setup it so for example 5 colabolators needs to review pull request for it to be accepted.