-
Notifications
You must be signed in to change notification settings - Fork 8
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
src: Add hot reload option for development #73
Conversation
✅ Deploy Preview for pulpito ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@VallariAg Sorry I closed the other one by accident, staging, and forgot to create a commit before force pushing lol Anyways, I got rid of the package-lock changes. |
@@ -20,6 +20,10 @@ export default defineConfig(() => { | |||
], | |||
server: { | |||
port: 8081, | |||
host: true, | |||
watch: { | |||
usePolling: true, |
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.
Everything looks good! Just one small thing: according to the docs, usePolling
uses high CPU utilisation so we should probably be using this for developer setup only and not production env? https://vitejs.dev/config/server-options#server-watch
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.
sounds good!
Allow hot reload in Development mode for pulpito-ng. This saves a lot more time when developing with other tech stacks. Signed-off-by: Kamoltat Sirivadhna <ksirivad@redhat.com>
c5b3262
to
a6f8888
Compare
@VallariAg updated the vite.config.js just need approval for merge |
Allow hot reload in Development mode for
pulpito-ng. This saves a lot more time
when developing with other tech stacks.