@pkozlowski-opensource this should also prevent Rx from being double-built into the bundles

The commit message should be fix(...) instead of chore.

Looks good with cleanup!

this surfaced an issue with RxJS typings on operators : ReactiveX/rxjs#635 - investigating. Means that this works fine:

http.get('/foo').subscribe(res => console.log(res.json());

but this complains:

http.get('/foo')
  .map(res => res.json()) //error: res (any) does not have property .json()
  .subscribe(foos => console.log(foos));

temporary workaround is to cast to response:

http.get('/foo')
  .map((res:Response) => res.json()) //ok
  .subscribe(foos => console.log(foos))

which kind of defeats the purpose 😿

@jeffbcross issues addressed, reordered the commits. issue above is non-blocking I think, so lets merge to get @pkozlowski-opensource unblocked on bundle stuff. give this a once over if you would.

Merging PR #5017 on behalf of @jeffbcross to branch presubmit-jeffbcross-pr-5017.

@robwormald merging!

Yay!

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.}