-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
cargotest: Add xsv to tested crates #38709
Conversation
This was intended to land in rust-lang#37149 but I ended up backing it out to land the rollup (rust-lang#38697) last night as I was itching to do so. This morning though xsv has been fixed now (BurntSushi/xsv#53) so we should be able to add it!
(rust_highfive has picked a reviewer for you, use r? to override) |
cc @brson this is something I'd expect to have gotten caught by cargobomb? |
r=me I guess? :) Commit seems fine, but I dno't know a lot about the plans here. |
@bors: r=nikomatsakis I'll take it! |
📌 Commit 73b708a has been approved by |
⌛ Testing commit 73b708a with merge 82fdb4e... |
💔 Test failed - status-travis |
@bors: retry
…On Fri, Dec 30, 2016 at 2:43 PM, bors ***@***.***> wrote:
💔 Test failed - status-travis
<https://travis-ci.org/rust-lang/rust/builds/187789726>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#38709 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAD95M62cvz62a1nowti9W4k_3ecigprks5rNYkMgaJpZM4LYUAq>
.
|
Yes cargobomb catches this. |
cargotest: Add xsv to tested crates This was intended to land in #37149 but I ended up backing it out to land the rollup (#38697) last night as I was itching to do so. This morning though xsv has been fixed now (BurntSushi/xsv#53) so we should be able to add it!
☀️ Test successful - status-appveyor, status-travis |
This was intended to land in #37149 but I ended up backing it out to land the
rollup (#38697) last night as I was itching to do so. This morning though xsv
has been fixed now (BurntSushi/xsv#53) so we should be able to add it!