Skip to content

Conversation

nodakai
Copy link
Contributor

@nodakai nodakai commented Feb 10, 2016

Just a small documentation change.

It would be great if anyone could check my English.

@rust-highfive
Copy link
Contributor

r? @aturon

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

/// capture local variables, and macros are only allowed in item,
/// statement or expression position, meaning this macro may be difficult to
/// use in some situations.
/// capture local variables. Also, as a general rule, macros are only
Copy link
Contributor

Choose a reason for hiding this comment

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

one space after a period please!

@steveklabnik
Copy link
Contributor

Your English is great! Just a small nit about the spaces and let's :shipit:

@nodakai
Copy link
Contributor Author

nodakai commented Feb 10, 2016

Thanks, corrected to a single space style.

@steveklabnik
Copy link
Contributor

@bors: r+ rollup

On Feb 10, 2016, 15:26 -0500, Kai [email protected], wrote:

Thanks, corrected to a single space style.


Reply to this email directly orview it on GitHub(#31542 (comment)).

@bors
Copy link
Collaborator

bors commented Feb 10, 2016

📌 Commit cec158b has been approved by steveklabnik

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Feb 13, 2016
…klabnik

Just a small documentation change.

It would be great if anyone could check my English.
Manishearth added a commit to Manishearth/rust that referenced this pull request Feb 13, 2016
…klabnik

Just a small documentation change.

It would be great if anyone could check my English.
bors added a commit that referenced this pull request Feb 13, 2016
@bors bors merged commit cec158b into rust-lang:master Feb 14, 2016
@nodakai nodakai deleted the concat_idents-desc branch March 8, 2016 05:03
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.

5 participants