-
Notifications
You must be signed in to change notification settings - Fork 361
fix(build): update vite #4570
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
fix(build): update vite #4570
Conversation
👷 Deploy request for vuestic-docs pending review.Visit the deploys page to approve it
|
✅ Deploy Preview for vuestic-storybook ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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 updates the build configuration to use Vite v7.1.1 and modernizes the SCSS module system by migrating from @import to the new Sass @use and @forward syntax. This update aligns with current best practices and prepares the codebase for future Sass deprecations.
- Upgraded Vite from v4.5.14 to v7.1.1 and added ESM module support
- Replaced deprecated
@importstatements with@use/@forwardsyntax across all SCSS files - Updated build configuration and plugin compatibility for Vite 7
Reviewed Changes
Copilot reviewed 134 out of 137 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/ui/package.json | Updated Vite to v7.1.1, added ESM module type, replaced Rollup with Vite bundling |
| packages/ui/src/styles/**/*.scss | Migrated from @import to @use/@forward syntax with proper namespacing |
| packages/ui/src/components/**/*.vue | Updated SCSS imports to use @use with module namespacing |
| packages/ui/build/plugins/component-v-bind-fix.ts | Enhanced plugin for Vite 7 compatibility with better type safety |
| packages/ui/postcss.config.js | Converted to ESM export format |
No description provided.