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

run rustfmt on syntax::parse::lexer #30065

Closed
wants to merge 4 commits into from
Closed

run rustfmt on syntax::parse::lexer #30065

wants to merge 4 commits into from

Conversation

tshepang
Copy link
Member

No description provided.

@rust-highfive
Copy link
Collaborator

r? @brson

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

.skip(1)
.all(|c| c == '*') {
if j > i &&
lines[j - 1]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Downgrade here.

@nagisa
Copy link
Member

nagisa commented Nov 26, 2015

I also have a few nits for parse/mod.rs changes, but Github won’t show the file for me to comment on.

@bors
Copy link
Contributor

bors commented Dec 18, 2015

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

@brson
Copy link
Contributor

brson commented Dec 28, 2015

Waiting for rebase or rerun.

@tshepang
Copy link
Member Author

Too messy to do a rebase. Will do a re-run.

@tshepang
Copy link
Member Author

tshepang commented Jan 3, 2016

A new PR: #30684

@tshepang tshepang closed this Jan 3, 2016
@tshepang tshepang deleted the rustfmt-lexer branch January 3, 2016 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants