Skip to content

chore: enable React hook rules part 2 and fix validation hooks - #216

Draft
aminamos wants to merge 1 commit into
MobilityData:mainfrom
aminamos:feat/164-eslint-react-hooks-2
Draft

chore: enable React hook rules part 2 and fix validation hooks#216
aminamos wants to merge 1 commit into
MobilityData:mainfrom
aminamos:feat/164-eslint-react-hooks-2

Conversation

@aminamos

Copy link
Copy Markdown

Summary:

Resolves #164.
Enables React hook rules (rules-of-hooks, exhaustive-deps, immutability) in eslint config and resolves hook dependency warnings in validation report components.

Expected behavior:

No React hook lint errors or unstable dependency warnings during linting.

Testing tips:

  1. Checkout feat/164-eslint-react-hooks-2.
  2. Run yarn lint to confirm zero hook violations.

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Run the unit tests with yarn test to make sure you didn't break anything
  • Add or update any needed documentation to the repo
  • Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
  • Linked all relevant issues
  • Include screenshot(s) showing how this pull request works and fixes the issue(s)

@vercel

vercel Bot commented Sep 14, 2026

Copy link
Copy Markdown

@aminamos is attempting to deploy a commit to the MobilityData Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enable React hook rules part 2

2 participants