fisker Cheung

Results 1279 comments of fisker Cheung

This was changed in https://github.com/sindresorhus/eslint-plugin-unicorn/pull/878, I thought it's correct, maybe we should stop fixing it if typeAnnotation found?

[`object-shorthand`](https://eslint.org/docs/rules/object-shorthand)?

Sorry for the delay, I'll review ASAP.

`no-suspicious-parentheses`?

I think we should only check built-in functions that doesn't accept a `boolean`.

> Should I check if they are in the same scope? Shouldn't they next to each other?

This is how I did in other rule https://github.com/sindresorhus/eslint-plugin-unicorn/blob/a7f600796d53d67553959d6d1aa8b0993abd2633/rules/no-array-push-push.js#L22

@Slowlife01 I made some changes. I hope it's easier to understand the code. Mainly changes: 1. Instead of grouping calls. It only checks two sibling calls. 2. Relax the node...