Skip to content

Conversation

topecongiro
Copy link
Contributor

@topecongiro topecongiro commented Aug 12, 2017

Currently the span for ast::Local does not include the let keyword. This PR fixes it.

@petrochenkov
Copy link
Contributor

Thanks!
@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Aug 12, 2017

📌 Commit 64503b6 has been approved by petrochenkov

@alexcrichton alexcrichton added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Aug 12, 2017
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Aug 13, 2017
…rochenkov

Include 'let' keyword to the span for ast::Local

Currently the span for `ast::Local` does not the include the `let` keyword. This PR fixes it.
bors added a commit that referenced this pull request Aug 13, 2017
Rollup of 8 pull requests

- Successful merges: #43782, #43803, #43814, #43819, #43821, #43822, #43824, #43833
- Failed merges:
@bors bors merged commit 64503b6 into rust-lang:master Aug 13, 2017
@topecongiro topecongiro deleted the missing-span-let branch August 13, 2017 13:00
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.

4 participants