-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Replace tempdir by tempfile #50698
Replace tempdir by tempfile #50698
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
☔ The latest upstream changes (presumably #50235) made this pull request unmergeable. Please resolve the merge conflicts. |
☔ The latest upstream changes (presumably #50769) made this pull request unmergeable. Please resolve the merge conflicts. |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
Welp... Once rustfmt gets updated again, the tempdir dependency will disappear. |
📌 Commit 5a9e04b has been approved by |
@bors rollup |
☔ The latest upstream changes (presumably #50615) made this pull request unmergeable. Please resolve the merge conflicts. |
I rebased, bors didn't remove |
@bors r=michaelwoerister rollup- |
📌 Commit 8f27e6d has been approved by |
The reviewer is away, maybe someone from @rust-lang/compiler can review this? |
@bors r+ |
📌 Commit 1fec4af has been approved by |
🔒 Merge conflict |
@Eijebong you need to rebase on top of the latest master, even if you don't see any conflicts. Sorry for the trouble! |
This removes the last dependency on tempdir.
Done. Sorry for the delay |
@bors r=oli-obk |
📌 Commit c863049 has been approved by |
☀️ Test successful - status-appveyor, status-travis |
No description provided.