search

cujojs / most

Ultra-high performance reactive programming
MIT License
3.49k stars 206 forks source link

Consider renaming `each()` to `forEach()` #2

Closed briancavalier closed 10 years ago

briancavalier commented 10 years ago

For two related reasons:

  1. Consistency w/JS Array API, and
  2. Learnability

Even though I think the name forEach is not great, consistency should probably win. We could potentially provide an alias that is more "stream-like", but I haven't come up with anything I really like (maybe observe or consume, but that's another discussion)

briancavalier commented 10 years ago

Pinging @unscriptable and @fabricematrat to get their thoughts.

fabricematrat commented 10 years ago

+1 The different languages I know are using forEach for stream.

On Fri, Nov 29, 2013 at 4:51 PM, Brian Cavalier notifications@github.comwrote:

Pinging @unscriptable https://github.com/unscriptable and @fabricematrathttps://github.com/fabricematratto get their thoughts.

— Reply to this email directly or view it on GitHubhttps://github.com/cujojs/most/issues/2#issuecomment-29523938 .

Best regards,

http://rivieragug.org/ http://3musket33rs.github.com/

briancavalier commented 10 years ago

We can break the API in v0.3.0 if we decide to do this rename.