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.

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

Use Vite for dev builds - now with HMR! #7361

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

ivan-jukic
Copy link
Contributor

This PR introduces Vite as our solution for running dev builds. It provides an "out-of-the-box" hot module replacing, which should help us build those new UI features much quicker.

There is quite a few changes in this PR, but the gist is that they're quite relevant for making us faster.

There is also a number of "fixes", that align the codebase with the latest syntax, and some improvements like the ability to alias imports, which might be easier to reconcile than relative paths.


Screencast.From.2025-02-03.19-49-06.webm

Video shows random changes and how they are immediately reflected in the browser when using Vite+HMR This would take about 3+ minutes to compile and preview with our current setup.

Copy link

github-actions bot commented Feb 5, 2025

canbench 🏋 (dir: .)

./canbench_results.yml is up to date ✅


---------------------------------------------------

Benchmark: push_simple_text_messages
  total:
    instructions: 223.47 M (-0.20%) (change within noise threshold)
    heap_increase: 12 pages (no change)
    stable_memory_increase: 0 pages (no change)

---------------------------------------------------

Benchmark: add_reactions
  total:
    instructions: 891.68 M (-1.27%) (change within noise threshold)
    heap_increase: 3 pages (no change)
    stable_memory_increase: 0 pages (no change)

---------------------------------------------------

---
This PR introduces Vite as our solution for running dev builds. It provides an "out-of-the-box" hot module replacing, which should help us build those new UI features much quicker.

There is quite a few changes in this PR, but the gist is that they're quite relevant for making us faster.

There is also a number of "fixes", that align the codebase with the latest syntax, and some improvements like the ability to alias imports, which might be easier to reconcile than relative paths.
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