-
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
Add cargo check --all
#3731
Add cargo check --all
#3731
Conversation
r? @brson (rust_highfive has picked a reviewer for you, use r? to override) |
Thanks, this gives symmetry with |
src/bin/check.rs
Outdated
@@ -14,6 +14,7 @@ Usage: | |||
Options: | |||
-h, --help Print this message | |||
-p SPEC, --package SPEC ... Package to check | |||
--all Build all packages in the workspace |
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.
s/build/check/ ?
src/bin/check.rs
Outdated
@@ -14,6 +14,7 @@ Usage: | |||
Options: | |||
-h, --help Print this message | |||
-p SPEC, --package SPEC ... Package to check |
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.
There is ...
so both the help text and the flag name should be plural.
This'll check all `build` targets for all packages in a workspace
fc1f8d7
to
97c1bba
Compare
r+ from me :) |
@bors: r=matklad |
📌 Commit 97c1bba has been approved by |
Add `cargo check --all` This'll check all `build` targets for all packages in a workspace
☀️ Test successful - status-appveyor, status-travis |
This'll check all
build
targets for all packages in a workspace