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

--all-targets fixes #6039

Merged
merged 1 commit into from
Sep 18, 2018
Merged

--all-targets fixes #6039

merged 1 commit into from
Sep 18, 2018

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
Collaborator

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
Collaborator

bors commented Sep 17, 2018

📌 Commit 3a1cad6 has been approved by alexcrichton

@bors
Copy link
Collaborator

bors commented Sep 17, 2018

⌛ Testing commit 3a1cad6 with merge f376fae872289410bb3cc5baa43dd35fc368c200...

@bors
Copy link
Collaborator

bors commented Sep 17, 2018

💔 Test failed - status-appveyor

@alexcrichton
Copy link
Member

@bors: retry

@bors
Copy link
Collaborator

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
Collaborator

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