feat(cmds): #11 support running any cargo cmd(s) #18
Conversation
|
I'd prefer to have either the syntax The as what was proposed originally in #11. With this feature, it also makes no sense to have the Finally, add yourself to the Cargo.toml :) |
| @@ -54,3 +54,17 @@ pub fn run(cmd: &str) { | |||
| Err(e) => println!("Failed to execute 'cargo {}': {}", cmd, e) | |||
| }; | |||
| } | |||
|
|
|||
| /// Runs cargo by passing through a set of multiple commands and displays the output. | |||
| pub fn run_raw(cmds: &str) { | |||
There was a problem hiding this comment.
Given above comment above obsoleting the --<action> options, run_raw could simply replace run. Simply add a check (in compile.rs) that if cmd is empty or only whitespace, replace it with build test.
|
Given above comments, this will be a breaking change, bringing us to v3. |
|
Thanks for the comments! I agree that makes sense. I'll incorporate those changes. |
compressed
changed the title
|
I've updated the PR, much simpler logic now! |
Add support for running arbitrary cargo commands. For example, `cargo watch "test ex_ —release"` will run `cargo test ex_ —release`. Also supports multiple arguments: `cargo watch "build --release" "test test_"` will run `cargo build --release` followed by `cargo test test_`. If no arguments are given, default stays as-is: cargo build and cargo test. [breaking-change]
Support running any cargo command Closes #11.
|
Sorry for the wait, holidays :) |
|
No worries, thanks! |
Add support for running arbitrary cargo commands.
For example:
cargo watch "test ex_ —release"will run
cargo test ex_ —release.Also supports multiple arguments:
cargo watch build testwill run
cargo buildfollowed bycargo test.I've been using this for the last few days and haven't run into any issues.
Let me know any changes and I'll incorporate.