Skip to content
This repository has been archived by the owner on Apr 20, 2018. It is now read-only.

RxJS 5 support #16

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

RxJS 5 support #16

wants to merge 3 commits into from

Conversation

jadbox
Copy link

@jadbox jadbox commented Apr 25, 2016

Warning: this breaks RxJS 4

@LongLiveCHIEF
Copy link

bump!

@dignifiedquire
Copy link

any update on this?

@LongLiveCHIEF
Copy link

There are other problems with this lib that will require updates for RxJS 5 and node v4+, and also some of the primary functionality of this package can now be used more effectively with native RxJS 5 operators.

I get the feeling they're moving away from bringing this back into the main RxJS package, so I created a new project HackerHappyHour/rxjs-node aimed at adding node-based utilities to RxJS 5.

@QuentinRoy
Copy link

pausableBuffered is not implemented anymore. Hence, writeToStream won't work anymore..

@arichiardi
Copy link

Hello folks! I would be interested in knowing if there is anything to do for getting this in. A major version bump should make clear that compatibility with v4 is not guaranteed so I see no problem there.
If you really really want to make it clear, the versioning of this package could just jump to 5.x.x or something. This is a very valuable PR indeed!

@rightisleft
Copy link

Any update on this?

@QuentinRoy
Copy link

QuentinRoy commented Aug 23, 2017

The PR is incomplete as is so it is very unlikely to be merged until someone fixes it. But even then, I don't think this repository is still maintained. We'd probably need a fork (and a new maintainer).

@paulpdaniels
Copy link

Created a v5 port of this library here. Still kind of a work in progress, but I plan to try and keep it updated and I'll accept PRs.

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.

7 participants