-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Fix recommended run dev command in create-tauri-app #2555
Conversation
Co-authored-by: Lucas Nogueira <lucas@tauri.studio>
Co-authored-by: lucasfernog <lucasfernog@users.noreply.github.com>
Co-authored-by: lucasfernog <lucasfernog@users.noreply.github.com>
Co-authored-by: Ngo Iok Ui (Wu Yu Wei) <wusyong9104@gmail.com>
`--dev` is not a valid option for `tauri` cli; instead, we should use `-- dev` or just `dev`. Also, we can just use `tauri dev` for all package managers so we don't need to do the npm checking.
@wusyong could you please help me check this PR? 🙏 I've confirmed it locally and the screenshot is in the PR description. thank you! |
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.
Thanks for the PR, but please keep the double-dash when npm is used, because in some npm versions (I think npm v6), it will fail without it.
The fix should be to just add a space after the double-dash, so '--'
should be '-- '
Edit: I was wrong --
is only need when additional arguments like --ci
so your PR is fine
@amrbashir thank you so much for your confirmation! 🙏 and thanks for quick review from both of you!! |
--dev
is not a valid option fortauri
cli; instead, we should use-- dev
or justdev
. Also, we can just usetauri dev
for allpackage managers so we don't need to do the npm checking.
What kind of change does this PR introduce? (check at least one)
Does this PR introduce a breaking change? (check one)
The PR fulfills these requirements:
fix: #xxx[,#xxx]
, where "xxx" is the issue number)If adding a new feature, the PR's description includes:
Other information: