Skip to content

Rollup of 7 pull requests #28423

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

Closed
wants to merge 24 commits into from
Closed

Conversation

christopherdumas and others added 24 commits September 5, 2015 16:42
…nikomatsakis

Contributing to the Rust error explanations. Should I also add a better error for it by default?
@rust-highfive
Copy link
Contributor

r? @nrc

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

@steveklabnik
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Collaborator

bors commented Sep 15, 2015

📌 Commit d895106 has been approved by steveklabnik

@bors
Copy link
Collaborator

bors commented Sep 15, 2015

⌛ Testing commit d895106 with merge 7d46b64...

@bors
Copy link
Collaborator

bors commented Sep 15, 2015

💔 Test failed - auto-mac-64-opt

@alexispurslane
Copy link
Contributor

Can someone tell bors to retry?

@jonas-schievink
Copy link
Contributor

../src/libcoretest/atomic.rs:94:28: 94:37 error: wrong number of type arguments: expected 1, found 0 [E0243]
../src/libcoretest/atomic.rs:94     ensure_sync_and_send::<AtomicPtr>();
                                                           ^~~~~~~~~

Not a spurious failure. Looks like #28314

@Centril Centril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants