Uh oh!
There was an error while loading. Please reload this page.
implement the chalk-engine traits - #50937
Conversation
a2f0e10 to
5ef8cc9Comparerust-highfive
commented
May 21, 2018
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
fd52186Comparerust-highfive
commented
May 21, 2018
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
d5486daComparerust-highfive
commented
May 21, 2018
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 |
There was a problem hiding this comment.
Isn't it better to use an unimplemented!() here?
d5486da to
9dde1b2Comparerust-highfive
commented
May 22, 2018
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 |
There was a problem hiding this comment.
This will conflict with #50930, this file can be reverted to master version.
9dde1b2 to
70f0030Comparerust-highfive
commented
May 24, 2018
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
9ab03d9Comparerust-highfive
commented
May 24, 2018
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
b0a926fCompareb0a926f to
8fd316fComparenikomatsakis
commented
May 24, 2018
@bors delegate=scalexm,leodasvacas -- I figure this is pretty chalk-focused, so they're best qualified to r+ |
bors
commented
May 24, 2018
📌 Commit 8fd316f has been approved by |
bors
commented
May 24, 2018
✌️ @scalexm,leodasvacas can now approve this pull request |
scalexm
commented
May 24, 2018
@bors r+ |
bors
commented
May 24, 2018
@scalexm: 🔑 Insufficient privileges: Not in reviewers |
nikomatsakis
commented
May 24, 2018
@bors delegate=scalexm |
bors
commented
May 24, 2018
✌️ @scalexm can now approve this pull request |
nikomatsakis
commented
May 24, 2018
@bors r=scalexm |
scalexm
commented
May 24, 2018
@bors r+ |
bors
commented
May 24, 2018
💡 This pull request was already approved, no need to approve it again.
|
bors
commented
May 24, 2018
📌 Commit 8fd316f has been approved by |
bors
commented
May 24, 2018
💡 This pull request was already approved, no need to approve it again.
|
bors
commented
May 24, 2018
📌 Commit 8fd316f has been approved by |
bors
commented
May 24, 2018
⌛ Testing commit 8fd316f with merge 670f4dcb166188e86423f6e914ce6d14e556b6f5... |
bors
commented
May 24, 2018
💔 Test failed - status-travis |
rust-highfive
commented
May 24, 2018
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 |
rust-highfive
commented
May 24, 2018
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 |
nikomatsakis
commented
May 24, 2018
travis failures look spurious. |
emilyalbini
commented
May 24, 2018
That error is caused by travis itself (it wasn't able to download a ruby gem it needs to deploy things). @bors retry |
bors
commented
May 24, 2018
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
bors
commented
May 25, 2018
☀️ 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