Skip to content

Normalize spaces in lang attributes. #44900

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

Closed

Conversation

Havvy
Copy link
Contributor

@Havvy Havvy commented Sep 28, 2017

So, like, I grepped for all lang attributes for reasons and I noticed that they all share the same spacing of #[lang = "item_name"] except these five instances. So I decided to fix that. So enjoy this PR of exactly ten spaces.

@rust-highfive
Copy link
Contributor

r? @sfackler

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

@alexcrichton alexcrichton added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 28, 2017
@sfackler
Copy link
Member

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Sep 28, 2017

📌 Commit 20fc215 has been approved by sfackler

Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Sep 29, 2017
…ing, r=sfackler

Normalize spaces in lang attributes.

So, like, I grepped for all `lang` attributes for *reasons* and I noticed that they all share the same spacing of `#[lang = "item_name"]` except these five instances. So I decided to fix that. So enjoy this PR of exactly ten spaces.
bors added a commit that referenced this pull request Sep 30, 2017
@bors
Copy link
Collaborator

bors commented Sep 30, 2017

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

@Mark-Simulacrum
Copy link
Member

Merged in #44936.

@Havvy Havvy deleted the normalize-lang-attribute-spacing branch October 13, 2017 06:44
# 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.

6 participants