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

docs/readme: grammar and wording update #75

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

docs/readme: grammar and wording update #75

wants to merge 3 commits into from

Conversation

Bradley-Cooley
Copy link

Some of the English was a bit off in the readme. Updated a few pieces of it.

Some of the English was a bit off in the readme. Updated a few pieces of it.
@@ -72,7 +72,7 @@ cargo add leaf
[rust_getting_started]: https://doc.rust-lang.org/book/getting-started.html
[cargo-edit]: https://github.com/killercup/cargo-edit

If you are on a machine that doesn't have support for CUDA or OpenCL you
If you're on a machine that doesn't have support for CUDA or OpenCL you
Copy link
Member

Choose a reason for hiding this comment

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

I prefer the more formal "you are" without a contraction.

@hobofan hobofan self-assigned this Mar 9, 2016
@hobofan
Copy link
Member

hobofan commented Mar 9, 2016

@Bradley-Cooley Thanks! 👍

I made a few comments which I'd like to discuss before merging, the rest looks good. :)

Readme readability and grammar update based on previous commit's comments
@hobofan
Copy link
Member

hobofan commented Mar 10, 2016

@Bradley-Cooley Looks good now! Could you please squash the commits, and I will merge.

@homu
Copy link
Collaborator

homu commented Mar 14, 2016

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

@hobofan hobofan removed their assignment Mar 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants