Skip to content

Rework expected closure error #46800

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Dec 19, 2017

Conversation

estebank
Copy link
Contributor

  • point at def span
  • add label to primary span
  • use span_labels instead of span_notes

@rust-highfive
Copy link
Contributor

r? @arielb1

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

@estebank estebank force-pushed the expected-closure-def-span branch 2 times, most recently from 403ce74 to cce0a23 Compare December 18, 2017 07:13
@kennytm kennytm added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 18, 2017
* point at def span
* add label to primary span
* use `span_label`s instead of `span_note`s
@estebank estebank force-pushed the expected-closure-def-span branch from cce0a23 to 0555d25 Compare December 18, 2017 16:56
@arielb1
Copy link
Contributor

arielb1 commented Dec 18, 2017

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Dec 18, 2017

📌 Commit 0555d25 has been approved by arielb1

GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Dec 18, 2017
… r=arielb1

Rework expected closure error

* point at def span
* add label to primary span
* use `span_label`s instead of `span_note`s
bors added a commit that referenced this pull request Dec 19, 2017
Rollup of 11 pull requests

- Successful merges: #46700, #46786, #46790, #46800, #46801, #46802, #46804, #46805, #46812, #46824, #46825
- Failed merges:
@bors bors merged commit 0555d25 into rust-lang:master Dec 19, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
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.

5 participants