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

Remove unnecessary article #31612

Merged
merged 1 commit into from
Feb 14, 2016
Merged

Remove unnecessary article #31612

merged 1 commit into from
Feb 14, 2016

Conversation

raindev
Copy link
Contributor

@raindev raindev commented Feb 12, 2016

I feel sorry for bothering you with such a literally one character changes. If it is counter productive feel free to point it out in the comments, that would be totally understandable. I could try to pack such a changes together in one PR to make them less distractive.

r? @steveklabnik

@steveklabnik
Copy link
Member

I feel sorry for bothering you with such a literally one character changes.

Please do not! I love lots of tiny pull requests, they are much easier to review.

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Feb 12, 2016

📌 Commit eb0f9f8 has been approved by steveklabnik

@steveklabnik
Copy link
Member

(Also, this is larger than a single-character change: it's two characters! 😄 )

@raindev
Copy link
Contributor Author

raindev commented Feb 13, 2016

Thanks!

Now I see, this pull request is actually two times larger 😄

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Feb 13, 2016
I feel sorry for bothering you with such a literally one character changes. If it is counter productive feel free to point it out in the comments, that would be totally understandable. I could try to pack such a changes together in one PR to make them less distractive.

r? @steveklabnik
Manishearth added a commit to Manishearth/rust that referenced this pull request Feb 13, 2016
I feel sorry for bothering you with such a literally one character changes. If it is counter productive feel free to point it out in the comments, that would be totally understandable. I could try to pack such a changes together in one PR to make them less distractive.

r? @steveklabnik
bors added a commit that referenced this pull request Feb 13, 2016
@bors bors merged commit eb0f9f8 into rust-lang:master Feb 14, 2016
@raindev raindev deleted the grammar branch February 14, 2016 09:15
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.

3 participants