Effects.batch should complain if it doesn't get called with an array

[gordonbrander]

Currently calling Effects.batch(Effect, Effect, Effect) will fail silently. We should make this accidental call an exception or otherwise notify the developer.

[Gozala]

One thing to keep in mind is that flow type checker will catch that error. That being said if user does not use flow it's not going to help much. I need to find a way to report such errors without introducing perf tax to users with a type checker. The problem isn't free if (!Array.isArray(fx)) { throw ... }.

I could probably have reflex.noflow.js builds that would have those extra checks.

[gordonbrander]

@Gozala good point about the type checker. I'm not sure if its worth keeping this open.

[Gozala]

Closed as suggested in last comment.