microsoft / microsoft/fluentui

[Task] Enable eslint-plugin-react-hooks/lints/refs rules

Open Beginner friendly
#36,029 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Package: eslint-plugin
Dominant language
TypeScript
Stars
20.3k
Forks
2.9k
Avg merge
2d 9h
Merged PRs (30d)
46

Description

In order to enable the eslint-plugin-react-hooks/lints/refs we need to bump the eslint-plugin-react-hooks pacakge from 5.x to 7.x and that should be it, those rules should be enabled by default if we use the recommended preset

Originally posted by @layershifter in https://github.com/microsoft/fluentui/pull/35996#discussion_r3117548433

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Locate the package manifest and ESLint configuration used by the Fluent UI repository, then inspect how the recommended eslint-plugin-react-hooks preset is enabled. Bump eslint-plugin-react-hooks from 5.x to 7.x and verify that the refs rules are enabled by default without introducing lint failures.

Written by the indexing model from the issue text.

Assessment

Tech stack
react, typescript
Domain
developer-experience, tooling
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.