Skip to content

Conversation

@JackWilb
Copy link
Member

@JackWilb JackWilb commented Dec 15, 2022

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 change to the UI, it's all in the code.

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

TODOs:

  • Update relevant documentation
  • Fix trrack and trrack-vis to set "module": "dist/trrack.esm.js" and "module": "dist/trrackvis.esm.js", respectively, in their package.jsons.

@netlify
Copy link

netlify bot commented Dec 15, 2022

Deploy Preview for next-multilink ready!

Name Link
🔨 Latest commit 23a214e
🔍 Latest deploy log https://app.netlify.com/sites/next-multilink/deploys/63a0a92069a99d0008f980d3
😎 Deploy Preview https://deploy-preview-334--next-multilink.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.

@JackWilb JackWilb mentioned this pull request Dec 16, 2022
5 tasks
@JackWilb JackWilb merged commit 1ffc1bf into main Dec 19, 2022
@JackWilb JackWilb deleted the vite branch December 19, 2022 18:12
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