borkdude/speculative

Remove unnecessary s/cat with single component

Closed this issue · 2 comments

Example:

(s/fdef clojure.core//
  :args (s/cat :numbers (s/+ number?))
  :ret number?)

The s/cat here is unnecessary, just :args (s/+ number?) is sufficient.

What about:

(s/fdef clojure.core//
  :args (s/cat :numerator number?
               :denominators (s/* number?))
  :ret number?)

(this is in #50)

I like the :numerator :denominators approach