metosin/malli

More friendly explanation for `:malli.core/invalid-type`

Open

#750 opened on Sep 5, 2022

 (6 comments) (0 reactions) (0 assignees)Clojure (237 forks)batch import
enhancementhelp wanted

Repository metrics

Stars
 (1,724 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

Hi!

The following:

(malli.error/humanize
 (malli.core/explain [:sequential :string] #{"a"}))

returns "invalid-type". That could be improved in a couple ways:

  • it could say expected :sequential, got :set
    • i.e. mention the two specific types involved in the type mismatch.
  • additionally, or instead (for agility's sake), say there was a type mismatch for this value
    • while it might look like simply like a verbose rewording of "invalid type", it actually would make it evident that the value is wrong, not the schema itself (which is what I mistakenly believed while debugging this).

wdyt?

Thanks - V

Contributor guide