rxjs-tslint icon indicating copy to clipboard operation
rxjs-tslint copied to clipboard

fix mix

Open elvisbegovic opened this issue 7 years ago • 5 comments

For unknown reasons one file in my code contains this code (certainly months ago when update to pipeable manually I omit this ... it compile = i forget)

import "rxjs/add/operator/distinctUntilChanged";
...
this.form.get('iban').valueChanges.pipe(takeUntil(<Observable<any>> this.ngUnsubscribe))
      .distinctUntilChanged()
      .subscribe(this.onChange_iban.bind(this));

When running tslint my migrate-rxjs.tslint.json nothing change ^^ Is it relevant to fix this kind of code too?

elvisbegovic avatar Apr 11 '18 08:04 elvisbegovic

I've tried this locally and I didn't have an issue. Do you have a minimal file that reproduces the issue?

benlesh avatar Apr 12 '18 16:04 benlesh

if you compile this green cli6rc4 project issue6.zip with this rxjs-tslint and say it's normal behavior please close issue

FYI: all is inside app.component.ts

I was thinking here outdated will be removed and distinctUntilChange will move inside pipe() with correct path import but ok...

elvisbegovic avatar Apr 13 '18 14:04 elvisbegovic

I'm having the same issue that rxjs-5-to-6-migrate only converted the first operator to using pipe, but left all other chained operators untouched (which breaks the build due to changed imports)

Airblader avatar May 16 '18 08:05 Airblader

same issue for me. Im having to move all my .map operators by hand

geogramdotcom avatar Jun 24 '18 23:06 geogramdotcom

What ended up working for me was running the command multiple times.

Airblader avatar Jun 25 '18 04:06 Airblader