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

clean up comments to be more concise/uniform #1616

Merged
merged 1 commit into from
Oct 6, 2013
Merged

clean up comments to be more concise/uniform #1616

merged 1 commit into from
Oct 6, 2013

Conversation

bricejlin
Copy link
Contributor

made some changes to grammar/sentence structure

@parkr
Copy link
Member

parkr commented Oct 5, 2013

Thanks! It's curious that you opted out of "the" in these comments. Any particular reason for that?

@bricejlin
Copy link
Contributor Author

Could be a personal preference, but I find that when not necessary, leaving
out "the" gives people 1 less word to read while still being able to
extract the full meaning of a sentence. I don't like reading that many
'the's :P

@parkr
Copy link
Member

parkr commented Oct 6, 2013

Ok, cool. Thanks for the explanation!

parkr added a commit that referenced this pull request Oct 6, 2013
clean up comments to be more concise/uniform
@parkr parkr merged commit 75efbad into jekyll:master Oct 6, 2013
parkr added a commit that referenced this pull request Oct 6, 2013
@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants