Juho Teperi

It is possible to also use stream reading to read values from an array inside an object: https://github.com/metosin/jsonista/compare/stream-testing https://cassiomolin.com/2019/08/19/combining-jackson-streaming-api-with-objectmapper-for-parsing-json/ One just needs to navigate the parser to the array start...

Wrap-values is currently private, and that would be useful if a user wants to call e.g. `readValuesAs` themselves. Is that fn needed because the Iterators from Jackson don't implement Iterable...

The usual solution is to merge params to `:params` and add them under another key, e.g. `:path-params`, `:query-params`.

Btw. `wrap-params` doesn't only read query string parameters but also form parameters from request body, so it's quite common to use it.

@SevereOverfl0w Did you check `nrepl.cmdline` namespace? And there is also Java main class. Or is there some reason why `-e` would be better? ``` clj -Sdeps '{:deps {nrepl {:mvn/version "0.4.5"}}}'...

I have decided to release the next release using the old group-id and namespaces. I'd still like to eventually use some other group-id, but naming is hard.

I'm somewhat conflicted about using deep-merge here, it can potentially make it harder to undefine some options for production, which are set as default for development. We can probably prevent...

A small problem is that the metadata can't be provided when using command line options. But I guess that doesn't matter, if user needs to use metadata, one can add...

The change to option precedence already broke backwards compatibility so I don't think it would be too bad to use merge as default for maps. I think replace would be...

Need more info. I have not seen this.