search

MatAtBread / fast-async

603 stars 21 forks source link

Runtime included when not needed #16

Closed SimonDegraeve closed 7 years ago

SimonDegraeve commented 8 years ago

The runtime is inlined even when the file does not include any async call.

matAtWork commented 7 years ago

Just commit fast-async@6.1.0 to master which should address the issues you spotted (and another one from @rush about nodent and babel fighting over which one should map stack traces).

I'm not a big babel user, so if you want to pull this down and check it doesn't break anything you're doing, I'd appreciate any feedback before I npm publsh

SimonDegraeve commented 7 years ago

It looks good to me. Thank you.

SimonDegraeve commented 7 years ago

I notices few typo in the README

v6.1.x can also refernce the runtme via

And

The runtime need only be included once