implement the chalk-engine traits #50937
Conversation
a2f0e10 to
5ef8cc9
Compare
|
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 |
c9e1a26 to
fd52186
Compare
|
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 |
fd52186 to
d5486da
Compare
|
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 |
src/librustc_traits/chalk_context.rs
Outdated
There was a problem hiding this comment.
Isn't it better to use an unimplemented!() here?
d5486da to
9dde1b2
Compare
|
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 |
src/librustc/ty/mod.rs
Outdated
There was a problem hiding this comment.
This will conflict with #50930, this file can be reverted to master version.
9dde1b2 to
70f0030
Compare
|
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 |
70f0030 to
9ab03d9
Compare
|
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 |
9fc3dd1 to
b0a926f
Compare
b0a926f to
8fd316f
Compare
|
@bors delegate=scalexm,leodasvacas -- I figure this is pretty chalk-focused, so they're best qualified to r+ |
|
📌 Commit 8fd316f has been approved by |
|
✌️ @scalexm,leodasvacas can now approve this pull request |
|
@bors r+ |
|
@scalexm: 🔑 Insufficient privileges: Not in reviewers |
|
@bors delegate=scalexm |
|
✌️ @scalexm can now approve this pull request |
|
@bors r=scalexm |
|
@bors r+ |
|
💡 This pull request was already approved, no need to approve it again.
|
|
📌 Commit 8fd316f has been approved by |
|
💡 This pull request was already approved, no need to approve it again.
|
|
📌 Commit 8fd316f has been approved by |
|
⌛ Testing commit 8fd316f with merge 670f4dcb166188e86423f6e914ce6d14e556b6f5... |
|
💔 Test failed - status-travis |
|
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 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 |
|
travis failures look spurious. |
|
That error is caused by travis itself (it wasn't able to download a ruby gem it needs to deploy things). @bors retry |
implement the chalk-engine traits Preliminary implementation for the Chalk traits in rustc. Lots of `panic!()` placeholders to be filled in later. This is currently blocked on us landing rust-lang/chalk#131 in chalk and issuing a new release, which should occur later today. r? @scalexm cc @leodasvacas
|
☀️ Test successful - status-appveyor, status-travis |
Preliminary implementation for the Chalk traits in rustc. Lots of
panic!()placeholders to be filled in later.This is currently blocked on us landing rust-lang/chalk#131 in chalk and issuing a new release, which should occur later today.
r? @scalexm
cc @leodasvacas