Fork me on GitHub
#funcool
<
2016-08-30
>
mattly00:08:56

bide looks neat โ€“ย is there a reason it's cljs only?

niwinz04:08:02

@mattly Because I needed it for cljs. But I open to have it ported to clj ๐Ÿ˜‰

mccraigmccraig12:08:42

hey @niwinz i just hit a niggle with cats.core/when ... it's an fn rather than a macro, so any expression of the form (m/when test (throw-up-when-test-is-false)) always throws when test is false... so i tried rewriting when as a macro, but that still can't avoid evaluating the mv arg either, because it needs to call ctx/infer ... dyu have any ideas ?

mccraigmccraig12:08:07

i.e. unless a context is available from with-context it must be explicitly given to when or unless

dialelo13:08:44

makes sense to me @mccraigmccraig, we should change these to be macros because we don't want to evaluate the branch if the predicate doesn't hold

dialelo13:08:52

happy to merge a PR with these changes

mccraigmccraig13:08:55

cool, PR on the way @dialelo

dialelo13:08:50

thanks a lot!

dialelo13:08:17

sorry about the alet issues, i'm taking a look at it

dialelo13:08:50

my plan is to restrict what's possible in alet (for instance duplicate binding names) for making the code transformation more straightforward

mccraigmccraig13:08:33

i haven't found an alet issue i couldn't work around yet, so i'm still happy ๐Ÿ™‚