Skip to content

Print a line per downloaded crate #6085

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

Merged
merged 1 commit into from
Sep 24, 2018
Merged

Conversation

alexcrichton
Copy link
Member

This commit brings back the old one-line-per-crate output that Cargo's
had since the beginning of time but was removed in #6005. This was
requested on our users forum

This commit brings back the old one-line-per-crate output that Cargo's
had since the beginning of time but was removed in rust-lang#6005. This was
requested on our [users forum][1]

[1]: https://internals.rust-lang.org/t/testing-cargos-parallel-downloads/8466/2
@rust-highfive
Copy link

r? @matklad

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

@alexcrichton
Copy link
Member Author

r? @ehuss

@rust-highfive rust-highfive assigned ehuss and unassigned matklad Sep 24, 2018
@dtolnay
Copy link
Member

dtolnay commented Sep 24, 2018

I tried this locally and I am happy with the behavior. Thanks!

@ehuss
Copy link
Contributor

ehuss commented Sep 24, 2018

@bors r+

@bors
Copy link
Contributor

bors commented Sep 24, 2018

📌 Commit ef7e98f has been approved by ehuss

@bors
Copy link
Contributor

bors commented Sep 24, 2018

⌛ Testing commit ef7e98f with merge a0e3077...

bors added a commit that referenced this pull request Sep 24, 2018
Print a line per downloaded crate

This commit brings back the old one-line-per-crate output that Cargo's
had since the beginning of time but was removed in #6005. This was
requested on our [users forum][1]

[1]: https://internals.rust-lang.org/t/testing-cargos-parallel-downloads/8466/2
@bors
Copy link
Contributor

bors commented Sep 24, 2018

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

@bors bors merged commit ef7e98f into rust-lang:master Sep 24, 2018
@alexcrichton alexcrichton deleted the progress branch October 12, 2018 17:23
@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.

6 participants