metosin/malli

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

Open

#750 aperta il 5 set 2022

Vedi su GitHub
 (6 commenti) (0 reazioni) (0 assegnatari)Clojure (237 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (1724 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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

Guida contributor