Skip to content
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

Rename `--all` to `--workspace` #7241

Merged
merged 2 commits into from Sep 4, 2019

Conversation

@k-nasa
Copy link
Contributor

commented Aug 12, 2019

Background

close: #6977

Description

  • Warn when using the 'all' option
  • Changed help text
@rust-highfive

This comment has been minimized.

Copy link

commented Aug 12, 2019

r? @nrc

(rust_highfive has picked a reviewer for you, use r? to override)

@ehuss

This comment has been minimized.

Copy link
Contributor

commented Aug 12, 2019

Thanks! I think I'd like to avoid the warning for now. We can add it later on, and then people will be able to switch to the new flag (to avoid the warning), and still use older versions of Cargo.

Also, can you update the documentation in src/doc?

@k-nasa

This comment has been minimized.

Copy link
Contributor Author

commented Aug 13, 2019

Ok! I'll change src/doc, but I probably don't have time. May make you wait...

@k-nasa k-nasa force-pushed the k-nasa:all_to_workspace branch from 616f0d7 to 586683e Aug 13, 2019

@k-nasa

This comment has been minimized.

Copy link
Contributor Author

commented Aug 13, 2019

@ehuss I changed src/doc and delete warnning! How about?

@bors

This comment has been minimized.

Copy link
Contributor

commented Aug 19, 2019

☔️ The latest upstream changes (presumably #7270) made this pull request unmergeable. Please resolve the merge conflicts.

@k-nasa

This comment has been minimized.

Copy link
Contributor Author

commented Aug 22, 2019

I fixed conflict

@k-nasa k-nasa force-pushed the k-nasa:all_to_workspace branch from 457310c to 00ace3a Aug 22, 2019

@k-nasa

This comment has been minimized.

Copy link
Contributor Author

commented Aug 23, 2019

@ehuss ?

@ehuss

This comment has been minimized.

Copy link
Contributor

commented Aug 23, 2019

I will follow up once I've had a chance to discuss it with the team, I want to consider @est31's concern about the length of the option.

@ehuss ehuss force-pushed the k-nasa:all_to_workspace branch from 00ace3a to 2040b5b Sep 4, 2019

Update additional uses of --all.
- man pages
- Slightly reword deprecation notice.
- Include --all in man pages.
- Update some additional usages in code and docs.

@ehuss ehuss force-pushed the k-nasa:all_to_workspace branch from 2040b5b to ecf824f Sep 4, 2019

@ehuss

This comment has been minimized.

Copy link
Contributor

commented Sep 4, 2019

I added some updates. There were some parts that were missing updates (like the man pages). I also slightly reworded the help text. I also rebased on the latest master.

@bors r+

@bors

This comment has been minimized.

Copy link
Contributor

commented Sep 4, 2019

📌 Commit ecf824f has been approved by ehuss

bors added a commit that referenced this pull request Sep 4, 2019
Auto merge of #7241 - k-nasa:all_to_workspace, r=ehuss
Rename `--all` to `--workspace`

## Background

close: #6977

## Description
- Warn when using the 'all' option
- Changed help text
@bors

This comment has been minimized.

Copy link
Contributor

commented Sep 4, 2019

⌛️ Testing commit ecf824f with merge fe0e5a4...

@est31

This comment has been minimized.

Copy link
Contributor

commented Sep 4, 2019

@ehuss what happened to my suggestion?

@bors

This comment has been minimized.

Copy link
Contributor

commented Sep 4, 2019

☀️ Test successful - checks-azure
Approved by: ehuss
Pushing fe0e5a4 to master...

@bors bors merged commit ecf824f into rust-lang:master Sep 4, 2019

10 checks passed

homu Test successful
Details
rust-lang.cargo Build #20190904.5 succeeded
Details
rust-lang.cargo (Linux beta) Linux beta succeeded
Details
rust-lang.cargo (Linux nightly) Linux nightly succeeded
Details
rust-lang.cargo (Linux stable) Linux stable succeeded
Details
rust-lang.cargo (Windows x86_64-msvc) Windows x86_64-msvc succeeded
Details
rust-lang.cargo (docs) docs succeeded
Details
rust-lang.cargo (macOS) macOS succeeded
Details
rust-lang.cargo (resolver) resolver succeeded
Details
rust-lang.cargo (rustfmt) rustfmt succeeded
Details
@ehuss

This comment has been minimized.

Copy link
Contributor

commented Sep 4, 2019

@ehuss what happened to my suggestion?

I'd like to avoid using short flags unless it is an option that is heavily used. There are some options to reduce the burden of longer flags (shell completion, aliases, etc.). The --all flag will also stay forever, so if you really want to, you can continue to use it. We'd also like to better understand all the scenarios where this flag is even needed, and see if it can be avoided altogether in more situations. We can also always add a short flag at any time in the future.

@est31

This comment has been minimized.

Copy link
Contributor

commented Sep 4, 2019

In one of my multi-crate projects, cargo test only tests the default crate. I didn't realize this so one of my tests regressed. I only noticed much later. Then I switched to always doing cargo test --all manually after each commit. I'm not sure but I think I also need to do cargo check --all to be able to also check some binary targets that the default crate is using as library. Aliases are cool but personally I try to avoid customizing my computer as little as possible. But thanks for giving an explanation at least.

@k-nasa k-nasa deleted the k-nasa:all_to_workspace branch Sep 4, 2019

@ehuss ehuss referenced this pull request Sep 4, 2019
bors added a commit to rust-lang/rust that referenced this pull request Sep 4, 2019
Auto merge of #64150 - ehuss:update-cargo-books, r=alexcrichton
Update cargo, books

## cargo

8 commits in 22f7dd0495cd72ce2082d318d5a9b4dccb9c5b8c..fe0e5a48b75da2b405c8ce1ba2674e174ae11d5d
2019-08-27 16:10:51 +0000 to 2019-09-04 00:51:27 +0000
- Rename `--all` to `--workspace` (rust-lang/cargo#7241)
- Basic standard library support. (rust-lang/cargo#7216)
- Allow using 'config.toml' instead of just 'config' files. (rust-lang/cargo#7295)
- Retry on SSL Connect Error. (rust-lang/cargo#7318)
- minimal-copy `deserialize` for `InternedString` (rust-lang/cargo#7310)
- Fix typo in cargo vendor examples (rust-lang/cargo#7320)
- Fixes around multiple `[patch]` per crate (rust-lang/cargo#7303)
- Improve error messages on mkdir failure (rust-lang/cargo#7306)

## reference

7 commits in d191a0cdd3b92648e0f1e53b13140a14677cc65b..090c015f7939665866432c334957bd536c811870
2019-08-15 08:42:23 +0200 to 2019-09-03 13:59:28 -0700
- Fix rust-lang-nursery/reference#664: Review Oxford comma usage. (rust-lang-nursery/reference#668)
- Fix some links. (rust-lang-nursery/reference#667)
- Remove trait object warning. (rust-lang-nursery/reference#666)
- Specify pattern types in `let` statements and `for` expressions (rust-lang-nursery/reference#663)
- Fix loop expression link. (rust-lang-nursery/reference#662)
- async-await initial reference material (rust-lang-nursery/reference#635)
- Correct errors in the reference of extern functions definitions and declarations (rust-lang-nursery/reference#652)

## rust-by-example

1 commits in 580839d90aacd537f0293697096fa8355bc4e673..e76be6b2dc84c6a992e186157efe29d625e29b94
2019-08-17 23:17:50 -0300 to 2019-09-03 07:42:26 -0300
- Change link to russian translation repository (rust-lang/rust-by-example#1245)

## embedded-book

1 commits in 432ca26686c11d396eed6a59499f93ce1bf2433c..5ca585c4a7552efb546e7681c3de0712f4ae4fdc
2019-08-09 23:20:22 +0000 to 2019-08-27 13:39:14 +0000
- Fixup book CI  (rust-embedded/book#205)
@sharksforarms

This comment has been minimized.

Copy link

commented Sep 13, 2019

I also use --all on most of all my cargo commands since the project I work on is a workspace of over 5 modules/crates

Mainly these:
cargo build --all
cargo test --all
cargo fmt --all

Glad --all is staying, it's also shorter to type.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
7 participants
You can’t perform that action at this time.