Skip to content

Conversation

@Pixel998
Copy link
Contributor

@Pixel998 Pixel998 commented Nov 1, 2025

Prerequisites checklist

What is the purpose of this pull request?

Updates the ESLint configuration to use the latest eslint-plugin-react-hooks setup, which no longer includes the react-hooks/react-compiler rule.

What changes did you make? (Give an overview)

  • Upgraded eslint-plugin-react-hooks from ^6.0.0-rc.1 to ^7.0.1.
  • Removed the react-hooks/react-compiler rule, which was split into individual rules.
  • Switched to pluginReactHooks.configs.flat.recommended for a cleaner and up-to-date configuration.
  • Updated React and React DOM from 19.1.0 to 19.2.0.
  • Removed a no-longer-needed inline disable comment for react-hooks/react-compiler.

Related Issues

Closes #178

Is there anything you'd like reviewers to focus on?

@eslint-github-bot eslint-github-bot bot mentioned this pull request Nov 1, 2025
1 task
@eslintbot eslintbot added this to Triage Nov 1, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Nov 1, 2025
@netlify
Copy link

netlify bot commented Nov 1, 2025

Deploy Preview for eslint-code-explorer ready!

Name Link
🔨 Latest commit b1eab37
🔍 Latest deploy log https://app.netlify.com/projects/eslint-code-explorer/deploys/690625c8152c3e00084d4f55
😎 Deploy Preview https://deploy-preview-230--eslint-code-explorer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Member

@amareshsm amareshsm left a comment

Choose a reason for hiding this comment

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

LGTM. Leaving it open for 2nd review.

@amareshsm amareshsm moved this from Needs Triage to Second Review Needed in Triage Nov 1, 2025
@amareshsm amareshsm added the accepted There is consensus among the team that this change meets the criteria for inclusion label Nov 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted There is consensus among the team that this change meets the criteria for inclusion chore

Projects

Status: Second Review Needed

Development

Successfully merging this pull request may close these issues.

3 participants