multinet-app / multimatrix

A adjacency matrix visualization built to work with the multinet ecosystem.
https://multinet.app
Apache License 2.0
1 stars 0 forks source link

Migrate build and serve to use vite instead of using webpack #431

Closed JackWilb closed 1 year ago

JackWilb commented 1 year ago

Does this PR close any open issues?

No

Give a longer description of what this PR addresses and why it's needed

This speeds up our build step, serve step, and uses the current suggestions from vue for our build system. This will help with our migration to vue 3, pinia, and vuetify 3 when they are ready.

This switch to vite allows us to remove a ton of dependencies, such as babel, vue-cli, etc. This reduces the complexity of the app and the security risks of our dependencies.

These changes drops our build time from 30-60 to less than a second, 248ms on my last build. Incremental builds are also faster, but not as notably.

Provide pictures/videos of the behavior before and after these changes (optional)

No visual changes

Are there any additional TODOs before this PR is ready to go?

TODOs:

netlify[bot] commented 1 year ago

Deploy Preview for multimatrix ready!

Name Link
Latest commit 539141b3de3c5e02a4cadf87609e786ecbd26394
Latest deploy log https://app.netlify.com/sites/multimatrix/deploys/63a35eece096dd0009164569
Deploy Preview https://deploy-preview-431--multimatrix.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.