Skip to content

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
Contributor 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