Skip to content

Conversation

@ehuss
Copy link
Contributor

@ehuss ehuss commented Sep 17, 2018

  • Fix: cargo test --all-targets was running lib tests three times.
  • --all-targets help strings were wrong or misleading.
  • Minor cleanup to add Proposal type to maybe make the code more readable.

Closes #5178.

@rust-highfive
Copy link

r? @matklad

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

@bors
Copy link
Contributor

bors commented Sep 17, 2018

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

- Fix: `cargo test --all-targets` was running lib tests three times.
- `--all-targets` help strings were wrong or misleading.
- Minor cleanup to add `Proposal` type to maybe make the code more readable.
@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Sep 17, 2018

📌 Commit 3a1cad6 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Sep 17, 2018

⌛ Testing commit 3a1cad6 with merge f376fae872289410bb3cc5baa43dd35fc368c200...

@bors
Copy link
Contributor

bors commented Sep 17, 2018

💔 Test failed - status-appveyor

@alexcrichton
Copy link
Member

@bors: retry

@bors
Copy link
Contributor

bors commented Sep 18, 2018

⌛ Testing commit 3a1cad6 with merge 1b42a21...

bors added a commit that referenced this pull request Sep 18, 2018
--all-targets fixes

- Fix: `cargo test --all-targets` was running lib tests three times.
- `--all-targets` help strings were wrong or misleading.
- Minor cleanup to add `Proposal` type to maybe make the code more readable.

Closes #5178.
@bors
Copy link
Contributor

bors commented Sep 18, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 1b42a21 to master...

@bors bors merged commit 3a1cad6 into rust-lang:master Sep 18, 2018
@ehuss ehuss added this to the 1.31.0 milestone Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cargo test --all-targets does not work exactly as advertised

5 participants