Fork me on GitHub
#hoplon
<
2017-06-22
>
thedavidmeister03:06:00

@flyboarder oh ok nice, so it's an "opt in" feature at this point?

thedavidmeister03:06:21

@jjttjj don't you have to configure your dev tools to display debug level console logs?

flyboarder06:06:27

@thedavidmeister correct, runtime specs are always opt in

flyboarder06:06:45

Macro specs happen at expansion time

thedavidmeister06:06:32

yeah i think this additional structure would make things easier to test

thedavidmeister06:06:48

and just generally predict the behaviour of too

thedavidmeister06:06:59

but it also might introduce new boilerplate too

thedavidmeister06:06:45

like, using :toggle as an example, I can currently pass it any truthy value and not worry too much about it, in the new system i have to explicitly ensure that i'm passing true or false and even nil is not OK

flyboarder06:06:53

Yes you are correct, although just like the attribute providers, you can always override the defaults, also nil should always be used to prevent an attribute from being applied IMO

flyboarder23:06:32

@thedavidmeister I see what you mean about nil also not being ok, I'll refactor the specs to be nil-able.

thedavidmeister23:06:35

@flyboarder yeah, if i send through nil i don't want to have to cast it to false every time

flyboarder23:06:22

Right nil is always valid for the removal of the attribute