CodeChefVIT / CodeChefVIT/papers-codechef

unnecessary params in useEffect dependencies in filterContent.tsx

Aperta
#462 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
TypeScript
Stelle
20
Fork
64
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.