Remove RxJS as a dependency

[jayphelps]

Needing to pull in RxJS via RxEmber (or ember-cli-rx) seems very unnecessary and boatful. The current places it's used seem very easy to do without Rx and would save consumers significant file size.

[benlesh]

Needing to pull in RxJS via RxEmber (or ember-cli-rx) seems very unnecessary and boatful.

[jayphelps]

Hahahahahahahahahahahahahah im dying help call 911 hahahahahahaa

[SirZach]

freshly installing this addon also did not include Rx. I needed to do a npm install --save-dev ember-cli-rx and an bower install --save-dev rxjs for it to work.

[jayphelps]

@SirZach Yep, it's not here but it needs to be. PR to fix: #56

[benlesh]

This should be cleared up now. (The meta issue of rxjs not installing properly)

[jayphelps]

Technically it's not :p this was to remove RxJS as a dependency entirely of nf-graph since it's a rather large lib for what we're using it for. As discussed, it's OK to punt on this for now, until outside contributors care enough. But I'd prefer this ticket left open since it is indeed not ideal. I'll let it be your call, but that's IMO.

[benlesh]

Meh. Okay.

[jayphelps]

<3

[benlesh]

I'm closing this as it's not a priority for us, or seemingly anyone else at the moment. I'm happy to take PRs around it, of course.