isInt function refactoring

[0o001]

No description provided.

[NurimOnsemiro]

I wonder if "const x = parseFloat(value);" is also unnecessary.

[0o001]

I wonder if "const x = parseFloat(value);" is also unnecessary.

I assume it's unnecessary for the following reasons:

1- Function title
... isInt(value: any): value is string ...

2- Number("1s") -> NaN
parseFloat("1s") -> 1.0

Number("1n") -> NaN
parseFloat("1n") -> 1.0

However, if this is an undesired situation, yes, this part can be changed; it would be more appropriate for authorized developers to answer.

ioredis/test/unit/utils.ts

Line 52 in 9c17550

expect(utils.isInt("1s")).to.eql(false);