Gabriella439 / pipes

Compositional pipelines
BSD 3-Clause "New" or "Revised" License
487 stars 72 forks source link

Add loop function from MVC #152

Closed bosu closed 9 years ago

bosu commented 9 years ago

The only downside I can think of is loop being a very common name (e.g. first patch in the series). OTOH, Pipes.Prelude has other common names (e.g. map) and requires careful imports even now.

Gabriella439 commented 9 years ago

Yeah, I'd love to add this with two small modifications:

bosu commented 9 years ago

The rename is done.

IMHO, the long comment is pure gold :) I was unable to delete it, hence moved it into ListT section of the tutorial module.

If you still think we ought to delete it, I'll unhappily oblige :)

Gabriella439 commented 9 years ago

Alright, we can keep it, but it needs a few more edits, since some of the stuff in there is mvc-specific (like the references to "model" or the base StateT monad). I will go through the commit and point out the parts that need to be slightly changed.

bosu commented 9 years ago

Done.

Gabriella439 commented 9 years ago

Thanks! I really appreciate the time you took to transplant this. :)