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

Conversation

Projects
None yet
3 participants
@bricejlin
Copy link
Contributor

bricejlin commented Oct 5, 2013

made some changes to grammar/sentence structure

@parkr

This comment has been minimized.

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

This comment has been minimized.

Copy link
Contributor Author

bricejlin commented Oct 5, 2013

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

This comment has been minimized.

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

Merge pull request #1616 from bricejlin/patch-1
clean up comments to be more concise/uniform

@parkr parkr merged commit 75efbad into jekyll:master Oct 6, 2013

1 check passed

default The Travis CI build passed
Details

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.
You can’t perform that action at this time.