Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Migration from Webpack to Vite #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Migration from Webpack to Vite #1

wants to merge 1 commit into from

Conversation

ab-shafei
Copy link

Migration from Webpack to Vite

This PR migrates the build system from Webpack to Vite for improved development experience and better performance.

Changes

  • Replace webpack configuration with vite.config.js
  • Update SASS imports to use modern @use/@forward syntax
  • Fix font loading with correct paths and formats
  • Improve code splitting with consistent dynamic imports
  • Update asset handling to work with Vite's conventions

Benefits

  • Faster development server startup
  • Better hot module replacement
  • Simpler configuration
  • Modern SASS module system
  • Improved code splitting

Testing

  • Development server works (bun run dev)
  • Production build works (bun run build)
  • Assets load correctly (fonts, images)
  • All pages render as expected
  • Navigation works properly

Notes

  • Removed webpack-related dependencies
  • Updated scripts in package.json
  • Maintained all existing functionality

@ab-shafei
refactor: migrate from webpack to vite
67509f9 
- Replace webpack with vite for better dev experience
- Update asset handling for vite - Convert SASS imports to use @use/@forward
- Fix font loading with correct path and format
- Update dynamic imports for better code splitting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ab-shafei