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

convience method #1

Closed
wants to merge 1 commit into from
Closed

Conversation

jswhisperer
Copy link

Not perfect but this reloads when there are changes (still a bit slow) -k kill the process before making a new one, "." the directory to watch, -e exlude "dist/" then the command to run.. npm dev

Not perfect but this reloads when there are changes (still a bit slow) -k kill the process before making a new one, "." the directory to watch, -e exlude "dist/" then the command to run.. npm dev
@shubhamzanwar
Copy link
Member

shubhamzanwar commented Sep 22, 2020

Ah, this is a good thing to implement!
However, there's a way of detecting these changes with webpack and rebuilding automatically. I think it's been implemented in react-nodegui-starter. Will check that and add it here.
Until then, I'm opening an issue #2 for this :)

@shubhamzanwar
Copy link
Member

Sorry for accidentally closing that!

@jswhisperer
Copy link
Author

No problem, for sure I knew there was a better way!

@shubhamzanwar
Copy link
Member

Closing this for now.
I've created a pull request in the vue-loader repo to enable hot-reloading when the webpack target is node.
vuejs/vue-loader#1731

Once that's merged, we can implement HMR in vue-nodegui 🙂

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