CodeChefVIT / CodeChefVIT/papers-codechef

unnecessary params in useEffect dependencies in filterContent.tsx

Open
#462 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
20
Forks
64
PR merge metrics
No merged PRs in 30d

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

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.