Incorrectly imports constants as functions

[0x-r4bbit]

Before

import { never } from 'rxjs/observable/never

After

import { NEVER as never } from 'rxjs';

^ This shouldn't be.

[benlesh]

this should be:

import { NEVER } fromrxjs'``

and then usage is just:

NEVER (not never() or NEVER())

[mgechev]

This will require the development of a new rule in order to be properly solved. Not sure if I'll be able to work on this in the next a couple of weeks.

[cartant]

Something to watch out for is that empty can be passed a scheduler argument, so simply replacing the call with the constant could change behaviour.