malli 2022-06-13 | Slack Archive

Fork me on GitHub

ikitommi04:06:16

awesome! I’ll try to cut a release today (lot’s of small changes waiting)

ingesol08:06:39

I tested function instrumentation in CLJS recently, and it works pretty well with the pretty printer. One thing I noticed as a pain point was that I wanted to see the input args when an output value does not match the schema. Would that make sense to add? Or did I simply miss it?

ikitommi09:06:54

it’s not passed to the error reporter, should be a small change: https://github.com/metosin/malli/blob/master/src/malli/core.cljc#L2453

ikitommi09:06:16

issue and pr would be most welcome.

✅ 1

ikitommi09:06:51

no, the args are passed in

ikitommi09:06:10

just not used: https://github.com/metosin/malli/blob/master/src/malli/dev/pretty.cljc#L51-L58

ingesol21:06:09

Thanks. Was a quick fix, posted a PR