CodeChefVIT / CodeChefVIT/papers-codechef
unnecessary params in useEffect dependencies in filterContent.tsx
Ouverte
- Langage dominant
- TypeScript
- Étoiles
- 20
- Forks
- 64
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
according to [official react documentation](https://react.dev/reference/react/useState?utm_source=chatgpt.com)
"The set function has a stable identity, so you will often see it omitted from Effect dependencies, but including it will not cause the Effect to fire. If the linter lets you omit a dependency without errors, it is safe to do."
so lines 217:227 needs to be reassessed
Guide de contribution
Ouvrir le guide de contribution
Évaluation
Cette issue n'a pas encore été évaluée.