--{example,bin,bench,test} with no argument now lists all available targets #6505
Conversation
|
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon. If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes. Please see the contribution instructions for more information. |
…replaced max_values -> number_of_values
|
It seems that targets are listed in alphabetical order on my windows machine (and on Travis' OS X VM, too, apparently) but they're not ordered on Linux. What to do here? |
I think it should be fine to call |
|
One concern I have is that you now have to be careful to include the call to |
|
Thank you for that, it all makes a lot of sense. |
|
@bors: r=ehuss |
|
📌 Commit a51759c has been approved by |
--{example,bin,bench,test} with no argument now lists all available targets
```
cargo run --bin
error: "--bin" takes one argument.
Available binaries:
cargo
error: process didn't exit successfully: `target\debug\cargo.exe run --bin` (exit code: 101)
```
Previous PR: #5062
Closes #2548
Notes:
- `optional_opt` is a weird name, can someone come up with a better one?
- Should I call clap's `usage()` as well when printing the error message?
|
☀️ Test successful - status-appveyor, status-travis |
Update cargo 13 commits in 34320d212dca8cd27d06ce93c16c6151f46fcf2e..2b4a5f1f0bb6e13759e88ea9512527b0beba154f 2019-01-03 19:12:38 +0000 to 2019-01-12 04:13:12 +0000 - Add test for publish with [patch] + cleanup. (rust-lang/cargo#6544) - Fix clippy warning (rust-lang/cargo#6546) - Revert "Workaround by using yesterday's nightly" (rust-lang/cargo#6540) - Adding feature-flags to `cargo publish` and `cargo package` (rust-lang/cargo#6453) - Fix the Travis CI badge (rust-lang/cargo#6530) - Add helpful text for Windows exceptions like Unix (rust-lang/cargo#6532) - Report fix bugs to Rust instead of Cargo (rust-lang/cargo#6531) - --{example,bin,bench,test} with no argument now lists all available targets (rust-lang/cargo#6505) - Rebuild on mid build file modification (rust-lang/cargo#6484) - Derive Clone for TomlDependency (rust-lang/cargo#6527) - publish: rework the crates.io detection logic. (rust-lang/cargo#6525) - avoid duplicates in ignore files (rust-lang/cargo#6521) - Rustflags in metadata (rust-lang/cargo#6503) r? @alexcrichton
Previous PR: #5062
Closes #2548
Notes:
optional_optis a weird name, can someone come up with a better one?usage()as well when printing the error message?