-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
frontend:dev:watcher
should be executed after frontend:install
.
#1182
Comments
@stffabi @leaanthony |
I would be fine calling an install before starting the frontend watcher. To be honest I have no expertise with these integrations, as we currently use another approach for integrating the frontend into wails. If @leaanthony is fine calling an install there I could provide a PR for this. |
I just feel that if you're spinning out a watcher, then that should probably handle everything involved in watching. Is that how they work? I intend to move the svelte template to vite so I'll get a bit more insight into this. Thanks @stffabi for offering to do the install. I really want to understand the use cases here for external watcher and proxying to Dev servers like vite. Everyone wants this! |
Yeah, absolutely. Unfortunately I really have no expertise and I don't have any usecase for this. So I'm not really a help... |
I had the same feeling. |
@leaanthony PR=> #1214 |
Amazing! Thanks @misitebao |
@stffabi @leaanthony |
Fixed by #1216 |
Description
After creating the project, run
wails dev
for the first time. Whenfrontend:install
has not been executed,frontend:dev:watcher
will also be executed, and the system prompts lack of dependencies.To Reproduce
Steps to reproduce the behaviour:
wails cli
withmaster
branchwails init -n wails-vue -t https://github.com/misitebao/wails-template-vue
"frontend:dev:watcher": "npm run build:watch -w js",
towails.json
filewails dev
Expected behaviour
Both synchronous and asynchronous scripts should be executed after the
frontend:install
command. Otherwise there will be missing dependencies.Screenshots
System Details
Additional context
None
The text was updated successfully, but these errors were encountered: