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: Don't panic when failing to write .lock file #66657

Merged
merged 1 commit into from
Nov 24, 2019

Conversation

ollie27
Copy link
Member

@ollie27 ollie27 commented Nov 23, 2019

It can be treated like any other unexpected IO error.

I couldn't think of a good way to add a test for this unfortunately.

r? @GuillaumeGomez

It can be treated like any other unexpected IO error.
@GuillaumeGomez
Copy link
Member

Me neither... Let's get it in then. Thanks!

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Nov 23, 2019

📌 Commit 05ef20f has been approved by GuillaumeGomez

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Nov 23, 2019
Centril added a commit to Centril/rust that referenced this pull request Nov 23, 2019
…laumeGomez

rustdoc: Don't panic when failing to write .lock file

It can be treated like any other unexpected IO error.

I couldn't think of a good way to add a test for this unfortunately.

r? @GuillaumeGomez
bors added a commit that referenced this pull request Nov 23, 2019
Rollup of 5 pull requests

Successful merges:

 - #61351 (Stabilize cfg(doc))
 - #66539 (Point at type in `let` assignment on type errors)
 - #66655 (rustdoc: Mark `--extern-private` as unstable)
 - #66657 (rustdoc: Don't panic when failing to write .lock file)
 - #66673 (Move def collector from `rustc` to `rustc_resolve`)

Failed merges:

r? @ghost
@bors bors merged commit 05ef20f into rust-lang:master Nov 24, 2019
@ollie27 ollie27 deleted the rustdoc_flock_panic branch November 24, 2019 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants