fulcrologic / guardrails

Efficient, hassle-free function call validation with a concise inline syntax for clojure.spec and Malli
Eclipse Public License 2.0
240 stars 26 forks source link

Fix specs and emit specs from >def/>fdef unless explicitly disabled #6

Closed thenonameguy closed 4 years ago

thenonameguy commented 4 years ago

This way projects that pull in guardrails through a transitive deps (unconfigured/missing guardrails.edn) will still by default emit specs.

awkay commented 4 years ago

Cool, thanks for noticing the bug. Are you using been using this in a project of any size?

thenonameguy commented 4 years ago

I am perfomance optimizing the latest Pathom 2.3.0-DEV snapshot, which pulls in this dependency. The cljs.env mocking on the Clojure side doesn't work for me, but for now I just decided to comment that part of the code out in com.fulcrologic.guardrails.config. I'll probably open a patch for that once I get some more time.