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

doc: Fix grammar so sentence makes sense #815

Closed
wants to merge 3 commits into from
Closed

doc: Fix grammar so sentence makes sense #815

wants to merge 3 commits into from

Conversation

oskwazir
Copy link

Just added 'then it' because the phrasing did not seem correct.

Just added 'then it' because the phrasing did not seem correct.
@silverwind
Copy link
Contributor

The changed text doesn't seem correct to me either. How about removing the 'then' (and maybe add a comma in its place) ?

@Fishrock123
Copy link
Contributor

I agree , it or , then it would be better.

@Fishrock123 Fishrock123 added the doc Issues and PRs related to the documentations. label Feb 12, 2015
Omer Wazir added 2 commits February 12, 2015 10:01
Using just 'it' makes enough sense.
I forgot to replace "then" with a comma.
@oskwazir
Copy link
Author

I added a comma and took ‘then’ out so now we have , it

@silverwind
Copy link
Contributor

👍

Fishrock123 pushed a commit that referenced this pull request Feb 13, 2015
Just added ', it' because the phrasing did not seem correct.

PR-URL: #815
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
@Fishrock123
Copy link
Contributor

Thanks, landed in ba40942

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants