Synchronously calling next() will run cb() synchronously about async-each HOT 3 CLOSED

This module does not enforce asynchrony, it is compatible with it. It is consistent in its handling, and therefore does not inherently cause zalgo issues - although it certainly is possible as you've demonstrated for users to do so with it.

It's worth noting that https://github.com/caolan/async behaves exactly the same with its each method.

I spent a few minutes experimenting with this and prepared an implementation that does enforce async like you're requesting, but I've also confirmed it would be a major breaking change if incorporated. And I don't think that's what we want to do. I'm pushing it in a branch in case it's interesting to you.

from async-each.

If you remain concerned you could also read this section of async's readme https://github.com/caolan/async#synchronous-iteration-functions

from async-each.

I think the current implementation is reasonable. I wonder if it's worth mentioning in the readme?

Thanks for sharing your thoughts!

from async-each.