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

Use suggestions instead of notes ref mismatches #46256

Merged
merged 1 commit into from
Dec 2, 2017

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
Collaborator

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
Contributor

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
Contributor

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
Contributor

bors commented Nov 27, 2017

📌 Commit e457cbd has been approved by arielb1

@bors
Copy link
Contributor

bors commented Nov 29, 2017

⌛ Testing commit e457cbd with merge 033288f8bbe1831f2538bf7b356433d9e7cd147a...

@bors
Copy link
Contributor

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
Contributor

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
Contributor

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
Contributor

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