This is bad for several reasons. The consuming application is not aware of what is imported where and under which conditions. Also it constitutes a side effect which is generally to be avoided. The right way to do it is to import the operator as a function and use it standalone (e.g. map.call(myObservable, ...)), passing the Observable as an argument. An example can be found in the Angular code base:
The current code modifies the prototype of
Observableusing statements like here:https://github.com/shlomiassaf/angular2-modal/blob/3884dc50e11c2f5ba3664843a806bfb55949713e/src/angular2-modal/plugins/bootstrap/modal.ts#L1
import 'rxjs/add/operator/combineLatest';This is bad for several reasons. The consuming application is not aware of what is imported where and under which conditions. Also it constitutes a side effect which is generally to be avoided. The right way to do it is to import the operator as a function and use it standalone (e.g.
map.call(myObservable, ...)), passing theObservableas an argument. An example can be found in the Angular code base:https://github.com/angular/angular/blob/master/modules/%40angular/router/src/router.ts#L20
https://github.com/angular/angular/blob/master/modules/%40angular/router/src/router.ts#L685