Skip to content
This repository has been archived by the owner on Feb 18, 2022. It is now read-only.

Added pull-streams version of async problem #30

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kwijibo
Copy link

@kwijibo kwijibo commented Mar 20, 2017

Hi,
I've created a pull-stream version of the Async Problem. It's my first time using the pull-stream libraries, so there may be a neater way of writing it.

@davidchambers
Copy link
Contributor

Does all the code need to live in main, @kwijibo? I ask because we already have lots of examples of approaches which fail to separate the pure transformations from the impure code. 😜

@kwijibo
Copy link
Author

kwijibo commented Mar 20, 2017

You can compose and modularise pull-streams pretty well it seems - I managed to move some stuff out. It would be nice though, if there were something like a parachain in order to use the same file() pull stream in both steps. I'm not sure what I think of pull-streams as a solution (to this particular problem), I just wanted to get a feel for them.

@davidchambers
Copy link
Contributor

I appreciate the pull request, @kwijibo. I'd like to avoid reaching the point where there are dozens of examples which do not satisfy the requirements distracting from the one or two examples which do satisfy the requirements. As a result I'd like to leave this pull request open indefinitely, so that those interested in pull-stream may find it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants