Skip to content

use a multispan for MANY_SINGLE_CHAR_NAMES #3921

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

Merged
merged 1 commit into from
Apr 8, 2019

Conversation

euclio
Copy link
Contributor

@euclio euclio commented Apr 4, 2019

No description provided.

@euclio euclio force-pushed the single-char-names-multispan branch from dce8d99 to 127fa6b Compare April 4, 2019 21:12
@flip1995 flip1995 added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Apr 5, 2019
@euclio euclio force-pushed the single-char-names-multispan branch from 127fa6b to 0d50d44 Compare April 5, 2019 14:07
@flip1995
Copy link
Member

flip1995 commented Apr 5, 2019

Thanks, this looks really good!

Was there an Issue for this or was it just something you wanted to be improved?

There's currently a pile of PRs waiting to be merged once Clippy builds again. I just throw this PR on this pile. There should be nothing you have to do for this to get merged, but wait.

@euclio
Copy link
Contributor Author

euclio commented Apr 5, 2019

No issue, just a potential improvement that I spotted. Sounds good, thanks!

@flip1995
Copy link
Member

flip1995 commented Apr 8, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Apr 8, 2019

📌 Commit 0d50d44 has been approved by flip1995

@bors
Copy link
Contributor

bors commented Apr 8, 2019

⌛ Testing commit 0d50d44 with merge 4fdd113...

bors added a commit that referenced this pull request Apr 8, 2019
use a multispan for MANY_SINGLE_CHAR_NAMES
@bors
Copy link
Contributor

bors commented Apr 8, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: flip1995
Pushing 4fdd113 to master...

@bors bors merged commit 0d50d44 into rust-lang:master Apr 8, 2019
@euclio euclio deleted the single-char-names-multispan branch June 10, 2022 01:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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.

3 participants