-
Notifications
You must be signed in to change notification settings - Fork 7
Migration to Vite #110
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migration to Vite #110
Conversation
feat: upgrade to vite with react 16
Bumps [esbuild](https://github.com/evanw/esbuild) to 0.25.12 and updates ancestor dependency [vitest](https://github.com/vitest-dev/vitest/tree/HEAD/packages/vitest). These dependencies need to be updated together. Updates `esbuild` from 0.21.5 to 0.25.12 - [Release notes](https://github.com/evanw/esbuild/releases) - [Changelog](https://github.com/evanw/esbuild/blob/main/CHANGELOG-2024.md) - [Commits](evanw/esbuild@v0.21.5...v0.25.12) Updates `vitest` from 0.34.6 to 4.0.15 - [Release notes](https://github.com/vitest-dev/vitest/releases) - [Commits](https://github.com/vitest-dev/vitest/commits/v4.0.15/packages/vitest) --- updated-dependencies: - dependency-name: esbuild dependency-version: 0.25.12 dependency-type: indirect - dependency-name: vitest dependency-version: 4.0.15 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com>
…lti-b4298e4899 chore(deps): bump esbuild and vitest
There was a problem hiding this comment.
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 migrates the project from Create React App (CRA) to Vite as the build tool, addressing dependency management issues highlighted by dependabot. The migration includes configuration updates, build script changes, and the removal of .js extensions from import statements to align with ES module standards.
Key changes:
- Replaced CRA with Vite configuration and updated build/test tooling
- Removed
.jsextensions from all local module imports across the codebase - Migrated test setup from ReactDOM to @testing-library/react
Reviewed changes
Copilot reviewed 17 out of 26 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| vite.config.js | Added Vite configuration with React plugin, JSX support, and test environment setup |
| package.json | Replaced react-scripts with vite/vitest dependencies and updated npm scripts |
| index.html | Created new HTML entry point with script reference to src/index.jsx |
| src/setupTests.js | Added jest-dom setup for testing environment |
| src/App.test.jsx | Migrated test from ReactDOM.render to @testing-library/react |
| src/App.test.js | Removed old CRA-style test file |
| src/*.jsx | Removed .js extensions from import statements across all component files |
| .github/dependabot.yml | Added dependabot configuration for weekly npm updates |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Removed CRA related comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 17 out of 26 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
The migration to the Vite build tool was triggered by dependabot PRs that could not easily be fixed on the original and now obsolete Create React App (CRA) build tool.