Skip to content

Fiddle: update vite react package to the correct one#122

Merged
krishan711 merged 1 commit into
mainfrom
vitereact
Mar 16, 2026
Merged

Fiddle: update vite react package to the correct one#122
krishan711 merged 1 commit into
mainfrom
vitereact

Conversation

@krishan711

Copy link
Copy Markdown
Contributor

Description

Checklist:

  • I have updated the CHANGELOG with a summary of my changes

Copilot AI review requested due to automatic review settings March 16, 2026 15:07
@krishan711 krishan711 merged commit cce51a2 into main Mar 16, 2026
5 checks passed
@krishan711 krishan711 deleted the vitereact branch March 16, 2026 15:10

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR replaces @vitejs/plugin-react-swc with @vitejs/plugin-react (v6.0.1), aligning the React Vite plugin with the correct/standard package. Since the new plugin's peer dependencies are properly satisfied (vite ^8.0.0), the --legacy-peer-deps flag is no longer needed and is removed from the makefile.

Changes:

  • Swapped @vitejs/plugin-react-swc for @vitejs/plugin-react in package.json and the vite config
  • Removed --legacy-peer-deps flags from makefile npm commands, now that peer dependencies are aligned
  • Updated package-lock.json accordingly, removing SWC-related transitive dependencies

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

File Description
scripts/react-app-vite/app.config.js Updated import and usage from plugin-react-swc to plugin-react
package.json Replaced @vitejs/plugin-react-swc dependency with @vitejs/plugin-react ^6.0.1
package-lock.json Regenerated lockfile reflecting the dependency swap and removal of SWC packages
makefile Removed --legacy-peer-deps flag from install/update/outdated commands

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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