style-guide

style-guide copied to clipboard

Prevent spread `...obj` inside a `arr.reduce()`

6

I've seen this pattern used several times where the developer is using spread inside a reduce causing a O(N^2) algorithm when avoiding the reduce would lead to O(N). Example usage...

styfle

Add rule to ensure consistent TODO comments

2

We've seen varied approaches to this, and it would be good to standardise - as it will help us build tooling around this later.

mrmckeb

Add stylelint

1

This has been something we've wanted to do for a while. The proposal is to: - Start with the [recommended](https://github.com/stylelint/stylelint-config-recommended) rule set, or the stricter [standard](https://github.com/stylelint/stylelint-config-standard) rule set. - Require...

mrmckeb

Enable `no-implicit-any-catch`

4

This was a recommendation from @dglsparsons: > We have the `useUnknownInCatchVariables` setting enabled in our tsconfig (it’s part of the strict family). > > But it only applies to `try...

mrmckeb

Whilst you can't extend an EditorConfig, we could programatically create one for new projects. This could easily be a small script in bin, and could also configure Prettier for the...

mrmckeb

Error when linting after upgrade to v6

5

`pnpm lint` results in: ``` workflow-api:lint: Oops! Something went wrong! :( workflow-api:lint: workflow-api:lint: ESLint: 8.57.0 workflow-api:lint: workflow-api:lint: TypeError: Cannot destructure property 'parent' of 'node' as it is undefined. workflow-api:lint: Occurred...

neil-sola

supprt eslint 9

9

Hi, I am getting a lot of depedency warnings now that eslint has released 9. Do you have plans to support this in the near future? Is there way to...

trixobird

Hey I tried to turn 'react-hooks/exhaustive-deps': 'off', since i am switching to biome and it has its equivalent rule. but warning comes like 'warning Unused eslint-disable directive (no problems were...

anandabhi04