Warning for isEmpty in WebStorm about you-dont-need-lodash-underscore

[Cowjiang]

You can infer the parameter 'obj' to the 'any' type
like this:

const isEmpty = (obj: any) => [Object, Array].includes((obj || {}).constructor) && !Object.entries((obj || {})).length;

[Cowjiang]

Let's see the declaration of 'Array.includes':

interface Array<T> {
    /**
     * Determines whether an array includes a certain element, returning true or false as appropriate.
     * @param searchElement The element to search for.
     * @param fromIndex The position in this array at which to begin searching for searchElement.
     */
    includes(searchElement: T, fromIndex?: number): boolean;
}

And the declaration of 'Object.constructor':

interface Object {
    constructor: Function;
}

And Webstorm show this:

Array<ObjectConstructor | ArrayConstructor>.includes(searchElement: ObjectConstructor | ArrayConstructor, fromIndex?: number): boolean

That means Generics '' was identified as '<ObjectConstructor | ArrayConstructor>', and that's why Webstorm will show the warning 'Argument type Function is not assignable to parameter type ObjectConstructor | ArrayConstructor'.

Umm, this warning doesn't show in VS Code. And if you change the Javascript Version to Flow in Webstorm, it won't show too.

[straurob]

You can infer the parameter 'obj' to the 'any' type

This seems to be only valid for Typescript, right? Sorry, I forgot to mention that I'm using Vanilla JavaScript.

[straurob]

I'll check the WebStorm issues if there is something similar. It obviously looks more like a WebStorm thing.