Skip to content

Conversation

Valloric
Copy link
Contributor

@Valloric Valloric commented Apr 20, 2020

One or two tests became build-pass without the FIXME because they really
needed build-pass (were failing without it).

Helps with #62277


This change is Reviewable

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 20, 2020
@Valloric
Copy link
Contributor Author

The mingw-check timed out and appears to be a flake; not sure how to trigger a re-run...

One or two tests became build-pass without the FIXME because they really
needed build-pass (were failing without it).

Helps with rust-lang#62277
@nikomatsakis
Copy link
Contributor

@bors r+ rollup

Thanks @Valloric !

@bors
Copy link
Collaborator

bors commented Apr 28, 2020

📌 Commit 2155adb has been approved by nikomatsakis

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 28, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 28, 2020
Rollup of 5 pull requests

Successful merges:

 - rust-lang#71340 (Moving more build-pass tests to check-pass)
 - rust-lang#71456 (Use lib crate type for SGX in `fn build_auxiliary`)
 - rust-lang#71615 (share some common code for compile-time miri instances)
 - rust-lang#71637 (Minor formatting changes in `cfg-sanitize.md`)
 - rust-lang#71641 (Update books)

Failed merges:

r? @ghost
@bors bors merged commit 0144bf6 into rust-lang:master Apr 28, 2020
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.

4 participants