Skip to content

Disable ESLint refs rule and update lucide-react icon imports#754

Merged
jamespepper81 merged 4 commits into
mainfrom
dev
May 28, 2026
Merged

Disable ESLint refs rule and update lucide-react icon imports#754
jamespepper81 merged 4 commits into
mainfrom
dev

Conversation

@jamespepper81
Copy link
Copy Markdown
Contributor

No description provided.

claude and others added 4 commits May 28, 2026 21:51
…dev deps

eslint-plugin-react-hooks 7.1.1 introduced the `react-hooks/refs` rule
which false-positives on react-hook-form's standard handleSubmit pattern.
Disable the rule in line with other overly-aggressive react-hooks rules
already turned off (immutability, purity, preserve-manual-memoization,
set-state-in-effect).

Also ran `npm update` to pull in the latest dev dependency patch releases
including @babel/* 7.29.7, esbuild 0.27.7, typescript-eslint 8.60.0,
jose 6.2.3, and unrs-resolver 1.12.2 among others.

https://claude.ai/code/session_01KxGvjGfAkonx9FLzGnidZ6
Upgrade lucide-react to v1.17.0 and disable refs ESLint rule
Renames deprecated 0.x alias icons to their canonical lucide-react 1.x
equivalents across all source files. The package.json/lock bump to ^1.17.0
was already applied by Dependabot; this completes the migration by updating
imports and JSX usages so the codebase no longer depends on the compatibility
shim layer.

Renames applied:
- Loader2 → LoaderCircle
- AlertCircle → CircleAlert
- CheckCircle → CircleCheckBig
- CheckCircle2 → CircleCheck
- HelpCircle → CircleQuestionMark
- PlusCircle → CirclePlus
- ArrowUpCircle → CircleArrowUp
- ArrowDownCircle → CircleArrowDown
- SendHorizonal → SendHorizontal
- Edit2 → Pen
- TestTube2 → TestTubeDiagonal
- BarChart3 → ChartColumn
- LineChart → ChartLine
- CandlestickChart → ChartCandlestick
- PieChart (lucide) → ChartPie

https://claude.ai/code/session_01PnA1ju9XE5u3UXjMro97SY
Update Lucide icon imports to use current icon names
@jamespepper81 jamespepper81 merged commit 461c3bf into main May 28, 2026
8 checks passed
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.

2 participants