prefer-number-properties: Add checkNaN option

[gurgunday]

I think the NaN check should also be toggleable, because I use it like I use null (absence of an object), to represent an absence of a number; and since null is global, this setting lets me be more consistent

Thanks for maintaining this amazing plugin!

[gurgunday]

CI error not related I believe

[sindresorhus]

// @Richienb

[fisker]

Should the option be checkNaN or checkNan ?

[Richienb]

Could we maybe set it to false by default? (breaking)
I also think that NaN is better than Number.NaN.

[sindresorhus]

Could we maybe set it to false by default? (breaking)

👍

[gurgunday]

Not sure why this is failing, will try to take a look

[gurgunday]

Yeah, sorry y'all — when I turn it off by default, a bunch of tests are failing and even though I modify them to be like Infinity checks, there are errors

I do not have the time to look into it, so let's focus on adding this feature in this PR

See f544b63 to view my latest try

[Richienb]

I would like some examples in the docs

[Richienb]

I do not have the time to look into it, so let's focus on adding this feature in this PR

That's ok, we can change it in the next PR.

Added to the tracking issue

[gurgunday]

I added one more, what else would you like to see?

[gurgunday]

I can't request a review so just tagging 🙏

@sindresorhus