rxjs-no-compat not detecting rxjs-compat import for Observable.of

[DaanKeun]

rxjs-no-compat not detecting rxjs-compat import for Observable.of

[cartant]

I'm not sure what you mean. That regexp is for valid import locations and there is no valid observable location in v6 without rxjs-compat.

I'll have a look at the ignored import 'rxjs-compat/add/observable/of a little later.

[DaanKeun]

Ahh right, I see that now 🤦‍♂.

Then I'm not sure why it is ignoring it.
Our current workaround is to add 'rxjs-compat' into the import-blacklist which is working for now so no rush.

[cartant]

I added a test to attempt a reproduction of the issue, but the test passes, so ... 🤷‍♂
See 4c67962

[DaanKeun]

For the test should the import not be import 'rxjs-compat/add/observable/of';?
Or am I misunderstanding how these tests work?

[cartant]

You've misunderstood. You use the old-style imports when using rxjs-compat. You should never have rxjs-compat in the import location. If you have a look in node_modules/rxjs you will find files in the places for the old import locations and those re-direct to rxjs-compat.

[DaanKeun]

Ahhh ok cool, thanks for taking the time looking into this 😄 and sorry for any wasted time

[cartant]

No worries.