Sindre Sorhus

Results 4030 comments of Sindre Sorhus
trafficstars

I think we should ship with a default preset for this rule that covers a lot of built-ins.

Sounds useful. The only problem I can think of are the checks assuming the inverse case is not an integer. ```js if (Number.isInteger(x)) { } else { // Assumes this...

Accepted. We can just add it to `prefer-string-raw`.

Where are you testing the GIF? macOS Quick Look does not correctly respect the count. You must drag the GIF into a browser to properly test it.

Then it's more likely a problem with PowerPoint.

Rule name: `no-multiple-array-checks-in-logical-expressions`?

Going to pass on this. `parse()` is intentionally **lossless** so you can distinguish `?a` (present, `null`) from `?a=` (present, `''`) and from **absent**. Skipping at parse time would silently drop...

It should throw a user-friendly error.