Turn on react / react-hook eslinters to ensure consistency with EE

weaveworks / weave-gitops

Weave GitOps provides insights into your application deployments, and makes continuous delivery with GitOps easier to adopt and scale across your teams.

https://docs.gitops.weave.works/

Apache License 2.0

905 stars 151 forks source link

Turn on react / react-hook eslinters to ensure consistency with EE #4182

Open AlinaGoaga opened 8 months ago

AlinaGoaga commented 8 months ago

Closes #3457

Prior to merging this we will need to have remove-req-use-useAsyncFn merged in EE as the removal of useRequestState will break enterprise.

What changed?

As agreed in triage:

Introduced react/react-hook rules in eslint config to match EE's eslint config
Deactivated:

prop-types
display-name

Enforced:

exhaustive-deps
jsx-key
rules-of-hooks
no-children-prop
no-unescaped-entities
no-unknown-property