Bugfix/rfc 2451 rerebalance tests#64546
Conversation
|
Thanks @weiznich! This looks good to me. I think we might want to add some more tests that show what happens when fundamental is used in a few more positions. Something like:
Do you think you can add such tests? Feel free to add any further combinations that seem worth covering -- I see no reason not to "tile the space" here with tests in every combination we can think of. |
|
@nikomatsakis I've added some more tests. |
|
@bors r+ |
|
📌 Commit a9c38d9 has been approved by |
|
@bors r- -- wait, I see an error in CI |
|
r=me once CI is green :) |
|
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 CI is green. |
|
@bors r+ |
|
📌 Commit 31b3012 has been approved by |
…e-tests, r=nikomatsakis Bugfix/rfc 2451 rerebalance tests r? @nikomatsakis Fixes rust-lang#64412 Depends/Contains on rust-lang#64414 cc rust-lang#55437 and rust-lang#63599
…e-tests, r=nikomatsakis Bugfix/rfc 2451 rerebalance tests r? @nikomatsakis Fixes rust-lang#64412 Depends/Contains on rust-lang#64414 cc rust-lang#55437 and rust-lang#63599
…e-tests, r=nikomatsakis Bugfix/rfc 2451 rerebalance tests r? @nikomatsakis Fixes rust-lang#64412 Depends/Contains on rust-lang#64414 cc rust-lang#55437 and rust-lang#63599
|
@weiznich There should be an even number of renames. You still have a comma somewhere. |
src/test/ui/coherence/impl[t]-foreign[foreign[t],local]-for-foreign.rs
Outdated
Show resolved
Hide resolved
|
@bors r- ^-- |
|
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 |
cfe2554 to
a482d52
Compare
|
@ecstatic-morse Not necessarily, because not all of the tests are compile fail tests. |
a482d52 to
9249a73
Compare
|
@Centril Next try? |
|
@bors r=nikomatsakis rollup=never |
|
📌 Commit 9249a73 has been approved by |
…e-tests, r=nikomatsakis Bugfix/rfc 2451 rerebalance tests r? @nikomatsakis Fixes rust-lang#64412 Depends/Contains on rust-lang#64414 cc rust-lang#55437 and rust-lang#63599
…e-tests, r=nikomatsakis Bugfix/rfc 2451 rerebalance tests r? @nikomatsakis Fixes rust-lang#64412 Depends/Contains on rust-lang#64414 cc rust-lang#55437 and rust-lang#63599
Rollup of 6 pull requests Successful merges: - #64455 (Add Long error explanation for E0531) - #64546 (Bugfix/rfc 2451 rerebalance tests) - #64589 (Differentiate AArch64 bare-metal targets between hf and non-hf.) - #64763 (Add E0734 and its long explanation) - #64793 (Fix format macro expansions spans to be macro-generated) - #64799 (Fix double panic when printing query stack during an ICE) Failed merges: r? @ghost
…e-tests, r=nikomatsakis Bugfix/rfc 2451 rerebalance tests r? @nikomatsakis Fixes rust-lang#64412 Depends/Contains on rust-lang#64414 cc rust-lang#55437 and rust-lang#63599
|
@bors p=3 |
…nikomatsakis Bugfix/rfc 2451 rerebalance tests r? @nikomatsakis Fixes #64412 Depends/Contains on #64414 cc #55437 and #63599
|
☀️ Test successful - checks-azure |
r? @nikomatsakis
Fixes #64412
Depends/Contains on #64414
cc #55437 and #63599