-
Notifications
You must be signed in to change notification settings - Fork 84
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Break things (followup from #166) #168
Break things (followup from #166) #168
Conversation
a1bc431
to
3604505
Compare
3604505
to
d3c0dc7
Compare
@paldepind Are you ready to break things? |
Time for |
Does this mean we can also remove #37? |
@nordfjord I'm ready 😄 I'll take a good look this weekend! |
This looks good to me 🎉 |
What about releasing this as |
@paldepind If you ask my opinion, I would like we all to investigate end stream related issues, mostly #116 and cast @c-dante @nordfjord, and think about how to signalize errors as well. #35 #164 I also wanted to pull up an deferred emitting discussion, but havent enough «meat» to start that discussion. You see, when you do Last several months I was developing some build system, where flyd plays the same role that node streams plays in gulp. I'm ok with flyd, but I was forced to solve some very specific tasks, like async stream resolving, promise—stream, stream—promise bridges, value caching/backpessure. Some of them are definitely outside flyd's scope, but due solving them I was thinking that flyd is not ready for v1 yet, and I personally would love some time for changes to occur before things become more stable. |
This will resolve #167
And remove the functionality deprecated by #166
this means
flyd.chain
instead)