bigtestjs / server

All BigTest development has moved to https://github.com/thefrontside/bigtest
https://github.com/thefrontside/bigtest
2 stars 1 forks source link

♻Make Atom a (mostly) vanilla class #79

Closed cowboyd closed 4 years ago

cowboyd commented 4 years ago

The original implementation jumped through a lot of hoops to make sure all of the operations were closures, that were bound to the this context so that they could be used independently:

let { next } = atom;
yield next();

However, this was hard to read for very little benefit. If that's the behavior we want, we can always using something like auto-bind https://github.com/sindresorhus/auto-bind

jnicklas commented 4 years ago

🚢