chore(deps): update dependency eslint-plugin-react-hooks to v7#305
chore(deps): update dependency eslint-plugin-react-hooks to v7#305smartway-bot[bot] wants to merge 1 commit into
Conversation
459f049 to
0a29422
Compare
0a29422 to
5b3dfbe
Compare
5b3dfbe to
40fadf1
Compare
40fadf1 to
aa335b5
Compare
aa335b5 to
cf389e3
Compare
f62109a to
365b985
Compare
365b985 to
f01d960
Compare
f01d960 to
8d92f7f
Compare
8d92f7f to
1e48062
Compare
59dea69 to
797ea24
Compare
797ea24 to
732c34a
Compare
732c34a to
54e4a00
Compare
54e4a00 to
12ad562
Compare
12ad562 to
9f7ce59
Compare
📝 WalkthroughWalkthroughUpdates the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
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
⛔ Files ignored due to path filters (1)
package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (1)
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", |
There was a problem hiding this comment.
🧩 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).
This PR contains the following updates:
^4.6.0->^7.0.0Release Notes
facebook/react (eslint-plugin-react-hooks)
v7.1.1Compare Source
Note: 7.1.0 accidentally removed the
component-hook-factoriesrule, causing errors for users who referenced it in their ESLint config. This is now fixed.component-hook-factoriesrule for backwards compatibility. (@mofeiZ in #36307)v7.1.0Compare 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-effectdetection, improved ref validation, and more helpful error reporting.useEffectEventchecks in component syntax. (@jbrown215 in #35041)set-state-in-effectvalidation with fewer false negatives. (@jorge-cab in #35134, @josephsavona in #35147, @jackpope in #35214, @chesnokov-tony in #35419, @jsleitor in #36107)v7.0.1Compare Source
useEffectEventvalues as JSX props to guard against accidental propagation. (#34820 by @jf-eirinha)export =so eslint-plugin-react-hooks emits correct types for consumers in Node16 ESM projects. (#34949 by @karlhorky)configs.flatso theconfigsexport is always defined. (#34950 by @poteto)v7.0.0Compare Source
This release slims down presets to just 2 configurations (
recommendedandrecommended-latest), and all compiler rules are enabled by default.recommended-latest-legacyandflat/recommendedconfigs. The plugin now providesrecommended(legacy and flat configs with all recommended rules), andrecommended-latest(legacy and flat configs with all recommended rules plus new bleeding edge experimental compiler rules). (@poteto in #34757)v6.1.1Compare Source
Note: 6.1.0 accidentally allowed use of
recommendedwithout flat config, causing errors when used with ESLint v9'sdefineConfig()helper. This has been fixed in 6.1.1.recommendedconfig for flat config compatibility. Therecommendedconfig has been converted to flat config format. Non-flat config users should userecommended-legacyinstead. (@poteto in #34700)recommended-latestandrecommended-latest-legacyconfigs that include React Compiler rules. (@poteto in #34675)NoUnusedOptOutDirectivesrule. (@poteto in #34703)hermes-parserand dependency. (@poteto in #34719)@babel/plugin-proposal-private-methodsdependency. (@ArnaudBarre and @josephsavona in #34715)v6.1.0Compare 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.
recommendedpreset. Legacy config moved torecommended-legacy. (@michaelfaith in #32457)usewithin try/catch blocks. (@poteto in #34040)useEffectEventfunctions in arbitrary closures. (@jbrown215 in #33544)React.useEffectin addition touseEffectin rules-of-hooks. (@Ayc0 in #34076)react-hookssettings config option that to acceptadditionalEffectHooksthat are used across exhaustive-deps and rules-of-hooks rules. (@jbrown215) in #34497v6.0.0Compare Source
Accidentally released. See 6.1.0 for the actual changes.
v5.2.0Compare Source
do/whileloops (@tyxla in #31720)v5.1.0Compare Source
do/whileloops (@tyxla in #28714)asexpression (@mskelton in #31119)v5.0.0Compare Source
_Buttonor_componentare no longer valid. (@kassens) in #25162useActionStatestable. (@eps1lon in #29665)asexpression in callback. (@StyleShit in #28202)asexpressions in deps array. (@StyleShit in #28189)React.use()the same asuse(). (@kassens in #27769)use()lint to non-experimental. (@kassens in #27768)asexpressions. (@cpojer in #27590)useEffect(fn, undefined). (@kassens in #27525)useEventtouseEffectEvent. (@sebmarkbage in #25881)useEventfunctions in dependency lists. (@poteto in #25512)useEventreferences instead. (@poteto in #25319)RulesOfHookswithuseEventrules. (@poteto in #25285)v4.6.2Compare Source
v4.6.1Compare 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.
This PR has been generated by Renovate Bot.