-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
feat(tauri-cli): add plugin remove command #11322
Conversation
Package Changes Through 8be2ec3There are 9 changes which include tauri-utils with minor, tauri with minor, @tauri-apps/api with minor, tauri-runtime with minor, tauri-runtime-wry with minor, tauri-bundler with minor, tauri-cli with minor, @tauri-apps/cli with minor, tauri-macros with patch Planned Package VersionsThe following package releases are the planned based on the context of changes in this pull request.
Add another change file through the GitHub UI by following this link. Read about change files or the docs at github.com/jbolda/covector |
crates/tauri-cli/src/remove.rs
Outdated
"Builder::new(|pass| todo!()).build()" | ||
} else if plugin == "localhost" { | ||
"Builder::new(todo!()).build()" |
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.
tauri add
command, adds todo!()
because user are requested to add code here, so this code likely changes and so our check here won't work.
4a972b3
to
48cf511
Compare
Thank you |
Closes #10384
TODO: I would like help understanding the failing snapshot test. Is my change invalid? Or, does the snapshot just need to be updated?