Skip to content

Improved sentence #4976

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

Closed
wants to merge 3 commits into from
Closed

Improved sentence #4976

wants to merge 3 commits into from

Conversation

edsonmedina
Copy link
Contributor

No description provided.

@timglabisch
Copy link
Contributor

👍

@@ -101,7 +101,7 @@ to control the number of posts to display on the blog homepage:
parameters:
homepage.num_items: 10

If you ask yourself when the last time was that you changed the value of
If you ask yourself when was the last time you changed the value of
Copy link
Member

Choose a reason for hiding this comment

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

Ah, I don't love the new one either. The problem is the sentence is almost a question, but not really a question, so it's funny both ways. So how about something totally different:

If you've done something like this in the past, it's likely that you've in fact *never* actually needed
to change that value.

What do you think @edsonmedina?

Copy link
Member

Choose a reason for hiding this comment

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

👍

@xabbuh
Copy link
Member

xabbuh commented Mar 8, 2015

@edsonmedina Could you please make the suggested changes? :)

@edsonmedina
Copy link
Contributor Author

Done @xabbuh

@xabbuh
Copy link
Member

xabbuh commented Mar 8, 2015

@edsonmedina If you now could please change the line wrapping to wrap them after the first word that crosses the 72nd character, your pull request will be perfect. :)

@edsonmedina
Copy link
Contributor Author

@xabbuh check it now

@wouterj
Copy link
Member

wouterj commented Mar 8, 2015

It's perfect now. Thanks @edsonmedina!

@xabbuh
Copy link
Member

xabbuh commented Mar 8, 2015

👍

@weaverryan
Copy link
Member

Thanks Edson for the PR and working with us with the updates. It's very nice now!

weaverryan added a commit that referenced this pull request Mar 13, 2015
This PR was submitted for the master branch but it was merged into the 2.3 branch instead (closes #4976).

Discussion
----------

Improved sentence

Commits
-------

97dcda5 Update configuration.rst
9bc95c6 Update configuration.rst
bc9500b Readability
@weaverryan weaverryan closed this Mar 13, 2015
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.

5 participants