RxJS v6 release + support

[IgorMinar]

Work breakdown:

• public api cleanup
  • entry-point cleanup
    • refactor (ben) The big reorganization PR for v6 (WIP) ReactiveX/rxjs#3145 + Reorg2 ReactiveX/rxjs#3223
    • docs
    • backwards compatibility solution
    • forward migration transform
    • google3 LSC
  • public api typings cleanup
    • refactor refactor(types): adjust Subscription and Observable types to be easier to consume externally ReactiveX/rxjs#3328
    • backwards compatibility solution
    • forward migration transform
    • google3 LSC (if needed)
  • runtime api changes
    • [ ] move Observable.lift to separate function
    • [ ] abort controllers - required by WHATWG
    • [ ] async iterators implementation
    • [ ] improve stack trace
• size (depends on public api cleanup)
  • Webpack v4 "side-effects":false support
• productivity
  • move rxjs to bazel
• g3
  • migrate away from rxjs/add/operator
    • transform
    • google3 LSC
  • remove local mods
  • move g3 sync to copybara
  • bring v6 to g3
• misc issues to resolve
  • fromPromise no longer exported in master (v6), the replacement from creation function is too heavy weight ReactiveX/rxjs#3374 fromPromise creation method missing

[shyamal890]

Request to add work bullet point addressing #19705 issue (updateOn debounce support in input)

[IgorMinar]

@shyamal890 that's totally unrelated to rxjs (it's a forms feature request) and unfortunately something that will have to wait due to all the other changes going on. I expect that we'll take a closer look at forms after v6.0 is out.

[michaeljota]

I know that rxjs was in RC when integrated in Angular, but now that this is stable, is still the plan to integrate a new version of rxjs that is currently in RC? Or this will be postponed until rxjs@6 is stable?

[sarunint]

What is stable? I believe both Angular and RxJS are still in release candidate.

[michaeljota]

Angular have several stable releases, and I wonder if this is something the team will wait for before launching stable 6

[sarunint]

Current stable version of Angular, v5.x, depends on RxJS version 5.x, which is also stable.

I think RxJS and Angular will reach v6 stable at almost the same time. (Well, the RxJS lead is also on the Angular core team)

[michaeljota]

I know, but RxJS had a rewrite between Angular 2 and 4, and I'm almost sure that then Angular 2 was released, rxjs was added in a RC version, as the final was not stable.

Angular 6 was schedule for early April, and we are late (pun unintended). I wonder if waiting for rxjs going stable is one of the things keeping 6 to being release as well.

[MattiJarvinen]

RxJS 6 is released @michaeljota https://github.com/ReactiveX/rxjs/releases/tag/6.0.0

I bet this issue is solved within a week.

[gwelr]

With the release of both Angular 6 and Rxjs 6, documentation is no longuer accurate. I saw the the work breakdown at the top of this ticket, but with the importance of Observable and Rxjs in Angular, I believe it is important to update the documentation.

Eg: https://angular.io/guide/rx-library#observable-creation-functions is no longuer valid.

it should be:

import { from } from 'rxjs';

// Create an Observable out of a promise
const data = from(fetch('/api/endpoint'));
// Subscribe to begin listening for async result
data.subscribe({
 next(response) { console.log(response); },
 error(err) { console.error('Error: ' + err); },
 complete() { console.log('Completed'); }
});

[alfaproject]

I still can't get rid of rxjs-compat because zone.js depends on it.
@JiaLiPassion any ETA to refactor that?

[MattiJarvinen]

@alfaproject what is your current zone.js version?

[alfaproject]

0.8.26

@MattiJarvinen-BA this is the culprit https://github.com/angular/zone.js/blob/f22065eec207da887872cf29c0b041c9cc9261b2/lib/rxjs/rxjs-fake-async.ts#L9

[JiaLiPassion]

@alfaproject, I am doing this now, I will try to make a test version next week.

[mat3e]

@gwelr I was also confused after using fromPromise inside the application generated with Angular CLI. I've created a pull request for a documentation update.

[gwelr]

@mat3e congrats for your first PR. And it has already been reviewed. It seems your alias is considered redundant (at least for the documentation). EG:

import { from as from, Observable, of } from 'rxjs';

should be

import { from, Observable, of } from 'rxjs';

[mat3e]

Yes, the alias was fromPromise and I changed it 🤦‍♂️ it is already fixed.

[alfaproject]

@JiaLiPassion it's been 6 months and I'm still requiring rxjs-compat because of unit testing. Any progress on that?

[JiaLiPassion]

@alfaproject, it has been merged into zone.js master, but please wait for the next zone.js release.

[pkozlowski-opensource]

We've been on RxJS 6.x for a while so I guess we can close this general issue. Please open new, focused issue(s) if there is any remaining work to be done.

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}