Skip to content
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

Rollup of 5 pull requests #70481

Closed
wants to merge 11 commits into from
Closed

Conversation

Centril
Copy link
Contributor

@Centril Centril commented Mar 27, 2020

Successful merges:

Failed merges:

r? @ghost

estebank and others added 11 commits March 25, 2020 14:13
Some of the bound restriction structured suggestions were incorrect
while others had subpar output.
Tweak `suggest_constraining_type_param`

Some of the bound restriction structured suggestions were incorrect while others had subpar output.

The only issue left is a suggestion for an already present bound when dealing with `const`s that should be handled independently.

Fix rust-lang#69983.
suggest `;` on expr `mac!()` which is good as stmt `mac!()`

Fixes rust-lang#34421 by implementing @jseyfried's suggestion in rust-lang#34421 (comment).

r? @petrochenkov
…pe, r=estebank

non-exhastive diagnostic: add note re. scrutinee type

This fixes rust-lang#67259 by adding a note:
```
    = note: the matched value is of type &[i32]
```
to non-exhaustive pattern matching errors.

r? @varkor @estebank
Refactor type_of for constants

If I have to look at this function for a few hours I want it to at least look good.

r? @varkor
clarify hir_id <-> node_id method names

resolves 2 FIXME.

r? @eddyb
@Centril
Copy link
Contributor Author

Centril commented Mar 27, 2020

@bors r+ p=5 rollup=never

@bors
Copy link
Contributor

bors commented Mar 27, 2020

📌 Commit ae4bfd8 has been approved by Centril

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Mar 27, 2020
@Centril Centril added the rollup A PR which is a rollup label Mar 27, 2020
@bors
Copy link
Contributor

bors commented Mar 27, 2020

⌛ Testing commit ae4bfd8 with merge aa18c4e2d67eeee736e7be87684e30e0a92cb906...

@bors
Copy link
Contributor

bors commented Mar 27, 2020

💔 Test failed - checks-azure

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Mar 27, 2020
@Centril Centril closed this Mar 27, 2020
@Centril Centril deleted the rollup-4owzvz4 branch March 27, 2020 21:39
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 S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants