Closed zrzka closed 7 years ago
@zrzka That sounds like a plan - I'm going to do testing post-merge, by switching some of my functions over to use this - I'm not super worried about a potentially broken snapshot.
@moea just let me know when you'll publish snapshot and I'll test it too
@zrzka Thank you! 0.6.5-SNAPSHOT
now contains these changes, I'll update it again once I merge #67
@moea using 0.6.5-SNAPSHOT in one of our project along with new :env
support and it looks good so far.
What's done:
->cli-arg-value
as multimethodvalidate-fn-spec!
removed (VPC related), because it fails anyway if it's wrongWhat's left:
I left
:cljs-lambda :env
intact in this PR. This PR just adds support for:env
in:cljs-lambda :defaults
and in:cljs-lambda :functions
. And now ...:cljs-lambda :env
and new:env
, we're not going to break compatibility, they can coexist together for a while,:cljs-lambda :env
will be trashedJust let me know what do you think and what to do.
Part of #56.