Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

estebank
Copy link
Contributor

On type mismatch errors, use a suggestion when encountering minimal
differences in type differences due to refs, instead of a note.

@rust-highfive
Copy link
Contributor

r? @arielb1

(rust_highfive has picked a reviewer for you, use r? to override)

@kennytm kennytm added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 25, 2017
@arielb1
Copy link
Contributor

arielb1 commented Nov 25, 2017

@bors r+

@bors
Copy link
Collaborator

bors commented Nov 25, 2017

📌 Commit a55f181 has been approved by arielb1

@kennytm kennytm added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 25, 2017
@bors
Copy link
Collaborator

bors commented Nov 27, 2017

☔ The latest upstream changes (presumably #46168) made this pull request unmergeable. Please resolve the merge conflicts.

@estebank
Copy link
Contributor Author

@bors r=arielb1

@bors
Copy link
Collaborator

bors commented Nov 27, 2017

📌 Commit e457cbd has been approved by arielb1

@bors
Copy link
Collaborator

bors commented Nov 29, 2017

⌛ Testing commit e457cbd with merge 033288f8bbe1831f2538bf7b356433d9e7cd147a...

@bors
Copy link
Collaborator

bors commented Nov 29, 2017

💔 Test failed - status-travis

@kennytm
Copy link
Member

kennytm commented Nov 29, 2017

@bors retry — travis-ci/travis-ci#8821

arielb1 pushed a commit to arielb1/rust that referenced this pull request Nov 30, 2017
Use suggestions instead of notes ref mismatches

On type mismatch errors, use a suggestion when encountering minimal
differences in type differences due to refs, instead of a note.
@arielb1
Copy link
Contributor

arielb1 commented Nov 30, 2017

This appears to have a semantic conflict with #46249 (on the src/test/ui/suggestions/str-array-assignment.rs test).

@bors r-

@kennytm kennytm added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Nov 30, 2017
@kennytm kennytm added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 30, 2017
@kennytm kennytm removed the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Nov 30, 2017
On type mismatch errors, use a suggestion when encountering minimal
differences in type differences due to refs, instead of a note.
@estebank
Copy link
Contributor Author

estebank commented Dec 1, 2017

Rebased against latest master.

@bors r=arielb1

@bors
Copy link
Collaborator

bors commented Dec 1, 2017

📌 Commit 36faafe has been approved by arielb1

@estebank estebank added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Dec 1, 2017
@bors
Copy link
Collaborator

bors commented Dec 2, 2017

⌛ Testing commit 36faafe with merge b15a8ea...

bors added a commit that referenced this pull request Dec 2, 2017
Use suggestions instead of notes ref mismatches

On type mismatch errors, use a suggestion when encountering minimal
differences in type differences due to refs, instead of a note.
@bors
Copy link
Collaborator

bors commented Dec 2, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: arielb1
Pushing b15a8ea to master...

@bors bors merged commit 36faafe into rust-lang:master Dec 2, 2017
@estebank estebank deleted the suggest-deref branch November 9, 2023 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants