Skip to content

chore(deps): update dependency eslint-plugin-react-hooks to v7#305

Open
smartway-bot[bot] wants to merge 1 commit into
mainfrom
renovate/major-react-monorepo
Open

chore(deps): update dependency eslint-plugin-react-hooks to v7#305
smartway-bot[bot] wants to merge 1 commit into
mainfrom
renovate/major-react-monorepo

Conversation

@smartway-bot
Copy link
Copy Markdown
Contributor

@smartway-bot smartway-bot Bot commented Oct 18, 2024

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
eslint-plugin-react-hooks (source) ^4.6.0 -> ^7.0.0 age adoption passing confidence

Release Notes

facebook/react (eslint-plugin-react-hooks)

v7.1.1

Compare Source

Note: 7.1.0 accidentally removed the component-hook-factories rule, causing errors for users who referenced it in their ESLint config. This is now fixed.

  • Add deprecated no-op component-hook-factories rule for backwards compatibility. (@​mofeiZ in #​36307)

v7.1.0

Compare Source

This release adds ESLint v10 support, improves performance by skipping compilation for non-React files, and includes compiler lint improvements including better set-state-in-effect detection, improved ref validation, and more helpful error reporting.

v7.0.1

Compare Source

v7.0.0

Compare Source

This release slims down presets to just 2 configurations (recommended and recommended-latest), and all compiler rules are enabled by default.

  • Breaking: Removed recommended-latest-legacy and flat/recommended configs. The plugin now provides recommended (legacy and flat configs with all recommended rules), and recommended-latest (legacy and flat configs with all recommended rules plus new bleeding edge experimental compiler rules). (@​poteto in #​34757)

v6.1.1

Compare Source

Note: 6.1.0 accidentally allowed use of recommended without flat config, causing errors when used with ESLint v9's defineConfig() helper. This has been fixed in 6.1.1.

v6.1.0

Compare Source

Note: Version 6.0.0 was mistakenly released and immediately deprecated and untagged on npm. This is the first official 6.x major release and includes breaking changes.

  • Breaking: Require Node.js 18 or newer. (@​michaelfaith in #​32458)
  • Breaking: Flat config is now the default recommended preset. Legacy config moved to recommended-legacy. (@​michaelfaith in #​32457)
  • New Violations: Disallow calling use within try/catch blocks. (@​poteto in #​34040)
  • New Violations: Disallow calling useEffectEvent functions in arbitrary closures. (@​jbrown215 in #​33544)
  • Handle React.useEffect in addition to useEffect in rules-of-hooks. (@​Ayc0 in #​34076)
  • Added react-hooks settings config option that to accept additionalEffectHooks that are used across exhaustive-deps and rules-of-hooks rules. (@​jbrown215) in #​34497

v6.0.0

Compare Source

Accidentally released. See 6.1.0 for the actual changes.

v5.2.0

Compare Source

v5.1.0

Compare Source

v5.0.0

Compare Source

  • New Violations: Component names now need to start with an uppercase letter instead of a non-lowercase letter. This means _Button or _component are no longer valid. (@​kassens) in #​25162

v4.6.2

Compare Source

v4.6.1

Compare Source


Configuration

📅 Schedule: Branch creation - Between 12:00 AM and 06:59 AM, only on Monday ( * 0-6 * * 1 ) (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

@smartway-bot smartway-bot Bot added the dependencies Pull requests that update a dependency file label Oct 18, 2024
@smartway-bot smartway-bot Bot force-pushed the renovate/major-react-monorepo branch from 459f049 to 0a29422 Compare December 12, 2024 19:03
@smartway-bot smartway-bot Bot changed the title chore(deps): update dependency eslint-plugin-react-hooks to v5 chore(deps): update react monorepo (major) Dec 12, 2024
@smartway-bot smartway-bot Bot force-pushed the renovate/major-react-monorepo branch from 0a29422 to 5b3dfbe Compare December 12, 2024 20:02
@smartway-bot smartway-bot Bot changed the title chore(deps): update react monorepo (major) fix(deps): update react monorepo (major) Dec 23, 2024
@smartway-bot smartway-bot Bot changed the title fix(deps): update react monorepo (major) fix(deps): update react monorepo (major) - autoclosed Mar 17, 2025
@smartway-bot smartway-bot Bot closed this Mar 17, 2025
@smartway-bot smartway-bot Bot deleted the renovate/major-react-monorepo branch March 17, 2025 08:52
@smartway-bot smartway-bot Bot changed the title fix(deps): update react monorepo (major) - autoclosed fix(deps): update react monorepo (major) Mar 24, 2025
@smartway-bot smartway-bot Bot reopened this Mar 24, 2025
@smartway-bot smartway-bot Bot force-pushed the renovate/major-react-monorepo branch from 5b3dfbe to 40fadf1 Compare April 4, 2025 21:01
@smartway-bot smartway-bot Bot force-pushed the renovate/major-react-monorepo branch from 40fadf1 to aa335b5 Compare May 20, 2025 13:02
@smartway-bot smartway-bot Bot changed the title fix(deps): update react monorepo (major) chore(deps): update react monorepo (major) May 20, 2025
@smartway-bot smartway-bot Bot force-pushed the renovate/major-react-monorepo branch from aa335b5 to cf389e3 Compare August 5, 2025 10:00
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 5, 2025

🛡️ The security scan result : Repo and Config

@smartway-bot smartway-bot Bot force-pushed the renovate/major-react-monorepo branch 2 times, most recently from f62109a to 365b985 Compare August 25, 2025 04:40
@smartway-bot smartway-bot Bot force-pushed the renovate/major-react-monorepo branch from 365b985 to f01d960 Compare August 29, 2025 16:06
@smartway-bot smartway-bot Bot force-pushed the renovate/major-react-monorepo branch from f01d960 to 8d92f7f Compare September 16, 2025 09:08
@smartway-bot smartway-bot Bot changed the title chore(deps): update react monorepo (major) chore(deps): update dependency eslint-plugin-react-hooks to v5 Sep 16, 2025
@smartway-bot smartway-bot Bot force-pushed the renovate/major-react-monorepo branch from 8d92f7f to 1e48062 Compare October 8, 2025 22:03
@smartway-bot smartway-bot Bot changed the title chore(deps): update dependency eslint-plugin-react-hooks to v5 chore(deps): update dependency eslint-plugin-react-hooks to v6 Oct 8, 2025
@smartway-bot smartway-bot Bot force-pushed the renovate/major-react-monorepo branch 2 times, most recently from 59dea69 to 797ea24 Compare October 15, 2025 23:01
@smartway-bot smartway-bot Bot changed the title chore(deps): update dependency eslint-plugin-react-hooks to v6 chore(deps): update dependency eslint-plugin-react-hooks to v7 Oct 15, 2025
@smartway-bot smartway-bot Bot force-pushed the renovate/major-react-monorepo branch from 797ea24 to 732c34a Compare October 31, 2025 17:02
@smartway-bot smartway-bot Bot force-pushed the renovate/major-react-monorepo branch from 732c34a to 54e4a00 Compare November 17, 2025 17:36
@smartway-bot smartway-bot Bot force-pushed the renovate/major-react-monorepo branch from 54e4a00 to 12ad562 Compare December 17, 2025 14:29
@smartway-bot smartway-bot Bot force-pushed the renovate/major-react-monorepo branch from 12ad562 to 9f7ce59 Compare April 23, 2026 22:59
@smartway-bot smartway-bot Bot requested a review from a team as a code owner April 23, 2026 22:59
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 23, 2026

📝 Walkthrough

Walkthrough

Updates the eslint-plugin-react-hooks dev dependency from version ^4.6.0 to ^7.0.0, incorporating a major version upgrade of the React Hooks ESLint plugin into the project's linting configuration.

Changes

Cohort / File(s) Summary
Dependency Upgrade
package.json
Bumps eslint-plugin-react-hooks dev dependency from ^4.6.0 to ^7.0.0, updating the React Hooks ESLint ruleset.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • jonperron

Poem

🐰 A hop and a bound, dependencies align,
React Hooks rules now shine so fine,
Version seven brings order bright,
Linting paths set gleaming right,
Onward to code that's crisp and tight! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore(deps): update dependency eslint-plugin-react-hooks to v7' directly and accurately describes the main change—upgrading eslint-plugin-react-hooks from v4.6.0 to v7.0.0. It is specific, clear, and follows conventional commit formatting.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch renovate/major-react-monorepo

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@package.json`:
- Line 61: The project uses eslint-plugin-react-hooks v7 which requires Node.js
18+, but package.json lacks an engines.node constraint; update package.json to
add an "engines" field with "node": ">=18" so tools and CI respect the
requirement (edit the package.json top-level object to include the engines.node
entry referenced alongside eslint-plugin-react-hooks).
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 98ee0a9f-7ad8-440f-83c8-7ba6c13c28cb

📥 Commits

Reviewing files that changed from the base of the PR and between 55da96a and 9f7ce59.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json

Comment thread package.json
"eslint-plugin-prettier": "^5.1.3",
"eslint-plugin-react": "^7.34.1",
"eslint-plugin-react-hooks": "^4.6.0",
"eslint-plugin-react-hooks": "^7.0.0",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Verify whether the repo already enforces Node >=18 for this dependency upgrade.

set -euo pipefail

echo "== package.json engines =="
cat package.json | jq '.engines // "NO_ENGINES_FIELD"'

echo
echo "== Node versions pinned in common CI/workflow files (if present) =="
fd -HI '(^|/)(\.nvmrc|\.node-version|package\.json|.*\.yml|.*\.yaml)$' .github workflows . 2>/dev/null \
  | xargs -r rg -n --no-heading 'node-version|setup-node|^\s*"engines"|^\s*"node"\s*:'

echo
echo "== ESLint plugin versions in package.json =="
cat package.json | jq '.devDependencies | with_entries(select(.key|test("^eslint|react-hooks")))'

Repository: ZeroGachis/smartway-react-native-ui

Length of output: 1390


Add Node.js version constraint to ensure eslint-plugin-react-hooks v7 compatibility.

eslint-plugin-react-hooks v7 requires Node.js 18+, but the project does not declare this via engines.node in package.json. While most CI workflows pin to Node 18+, there is no guarantee for local development or the parameterized unit-tests workflow. Add "engines": { "node": ">=18" } to package.json to enforce the requirement consistently.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@package.json` at line 61, The project uses eslint-plugin-react-hooks v7 which
requires Node.js 18+, but package.json lacks an engines.node constraint; update
package.json to add an "engines" field with "node": ">=18" so tools and CI
respect the requirement (edit the package.json top-level object to include the
engines.node entry referenced alongside eslint-plugin-react-hooks).

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

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants