no-duplicates: allow duplicate if one is namespace and other not

[sveyret]

It is a syntax error to put both namespace and non namespace import on the same line, so allow it.
Fix issue #1538

[sveyret]

Note that this is a simple implementation which will not check:

import * as ns from './foo'
import './foo'

I'm currently working on something better…

[coveralls]

Coverage increased (+0.002%) to 97.837% when pulling fe5d6ea on sveyret:nsduplicate into b4d5fd3 on benmosher:master.

[coveralls]

Coverage increased (+0.002%) to 97.837% when pulling fe5d6ea on sveyret:nsduplicate into b4d5fd3 on benmosher:master.

[coveralls]

Coverage increased (+0.002%) to 97.837% when pulling fe5d6ea on sveyret:nsduplicate into b4d5fd3 on benmosher:master.

[coveralls]

Coverage increased (+0.002%) to 97.837% when pulling fe5d6ea on sveyret:nsduplicate into b4d5fd3 on benmosher:master.

[coveralls]

Coverage increased (+0.002%) to 97.837% when pulling f84d457 on sveyret:nsduplicate into 7e71b50 on benmosher:master.

[SCasarotto]

Note that this is a simple implementation which will not check:

import * as ns from './foo'
import './foo'

I'm currently working on something better…

@sveyret Did you ever circle back on this? I am seeing this issue come up with the following:

import firebase from 'firebase/app';
import 'firebase/auth';
import 'firebase/database';

and was trying to figure out if this was a mistake on my side or if I should create an issue.

EDIT: Created an issue so feel free to disregard. #2054

[sveyret]

I don’t really remember, but I don’t think I did anything since this PR. It means an evolution is needed.