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

Extract & re-use filter_targets in cargo_compile #6621

Merged
merged 1 commit into from
Feb 4, 2019

Conversation

dwijnand
Copy link
Member

@dwijnand dwijnand commented Feb 3, 2019

Also drop part of the docs in list_rule_targets, as that info is now on Proposal.

@rust-highfive
Copy link

r? @alexcrichton

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

src/cargo/ops/cargo_compile.rs Outdated Show resolved Hide resolved
src/cargo/ops/cargo_compile.rs Show resolved Hide resolved
Also drop part of the docs in `list_rule_targets`, as that info is now on `Proposal`.
@dwijnand dwijnand force-pushed the clean-cargo_compile branch from a1ce703 to df9ca87 Compare February 4, 2019 15:34
@dwijnand dwijnand changed the title Extract & re-use list_all_targets in cargo_compile Extract & re-use filter_targets in cargo_compile Feb 4, 2019
@ehuss
Copy link
Contributor

ehuss commented Feb 4, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Feb 4, 2019

📌 Commit df9ca87 has been approved by ehuss

@bors
Copy link
Contributor

bors commented Feb 4, 2019

⌛ Testing commit df9ca87 with merge 1c03264...

bors added a commit that referenced this pull request Feb 4, 2019
Extract & re-use filter_targets in cargo_compile

Also drop part of the docs in `list_rule_targets`, as that info is now on `Proposal`.
@bors
Copy link
Contributor

bors commented Feb 4, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: ehuss
Pushing 1c03264 to master...

@bors bors merged commit df9ca87 into rust-lang:master Feb 4, 2019
@dwijnand dwijnand deleted the clean-cargo_compile branch February 4, 2019 18:10
bors added a commit to rust-lang/rust that referenced this pull request Feb 10, 2019
Bump cargo to 865cb70

Merged PRs:

* Replace util::without_prefix with Path::strip_prefix rust-lang/cargo#6620
* keep track of crates that are whitelisted to be used even if yanked rust-lang/cargo#6611
* Fix default DYLD_FALLBACK_LIBRARY_PATH on MacOS. rust-lang/cargo#6625
* Bail when trying to run "test --doc --no-run" rust-lang/cargo#6628
* In cargo test's help, add that examples are built rust-lang/cargo#6619
* Extract & re-use filter_targets in cargo_compile rust-lang/cargo#6621
* Test cleanup: remove unnecessary with_status(0) rust-lang/cargo#6630
* Fix run's help message rust-lang/cargo#6631
* Some updates to bash completion. rust-lang/cargo#6644
* Introduce Source::download_now rust-lang/cargo#6637
* Switch from unused_imports to deprecated to test unfixable warnings rust-lang/cargo#6649
bors added a commit to rust-lang/rust that referenced this pull request Feb 11, 2019
Bump cargo to 865cb70

Merged PRs:

* Replace util::without_prefix with Path::strip_prefix rust-lang/cargo#6620
* keep track of crates that are whitelisted to be used even if yanked rust-lang/cargo#6611
* Fix default DYLD_FALLBACK_LIBRARY_PATH on MacOS. rust-lang/cargo#6625
* Bail when trying to run "test --doc --no-run" rust-lang/cargo#6628
* In cargo test's help, add that examples are built rust-lang/cargo#6619
* Extract & re-use filter_targets in cargo_compile rust-lang/cargo#6621
* Test cleanup: remove unnecessary with_status(0) rust-lang/cargo#6630
* Fix run's help message rust-lang/cargo#6631
* Some updates to bash completion. rust-lang/cargo#6644
* Introduce Source::download_now rust-lang/cargo#6637
* Switch from unused_imports to deprecated to test unfixable warnings rust-lang/cargo#6649
@ehuss ehuss added this to the 1.34.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.

5 participants