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

rustdoc: Suppress warnings/errors with --test #39354

Merged
merged 1 commit into from Jan 31, 2017

Conversation

alexcrichton
Copy link
Member

Threads spawned by the test framework have their output captured by default, so
for rustdoc --test threads this propagates that capturing to the spawned
thread that we now have.

Closes #39327

Threads spawned by the test framework have their output captured by default, so
for `rustdoc --test` threads this propagates that capturing to the spawned
thread that we now have.

Closes rust-lang#39327
@rust-highfive
Copy link
Collaborator

r? @steveklabnik

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

@alexcrichton
Copy link
Member Author

cc @nikomatsakis

@steveklabnik
Copy link
Member

Seems fine.

@alexcrichton
Copy link
Member Author

@bors: r=steveklabnik

@bors
Copy link
Contributor

bors commented Jan 30, 2017

📌 Commit e2ef630 has been approved by steveklabnik

@bors
Copy link
Contributor

bors commented Jan 30, 2017

⌛ Testing commit e2ef630 with merge 1d5d404...

@bors
Copy link
Contributor

bors commented Jan 30, 2017

💔 Test failed - status-travis

@alexcrichton
Copy link
Member Author

alexcrichton commented Jan 30, 2017 via email

@bors
Copy link
Contributor

bors commented Jan 31, 2017

⌛ Testing commit e2ef630 with merge cd692cc...

bors added a commit that referenced this pull request Jan 31, 2017
rustdoc: Suppress warnings/errors with --test

Threads spawned by the test framework have their output captured by default, so
for `rustdoc --test` threads this propagates that capturing to the spawned
thread that we now have.

Closes #39327
@bors
Copy link
Contributor

bors commented Jan 31, 2017

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

@bors bors merged commit e2ef630 into rust-lang:master Jan 31, 2017
@alexcrichton alexcrichton deleted the less-output branch February 11, 2017 18:38
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.

None yet

4 participants