-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Improved sentence #4976
Conversation
👍 |
@@ -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 |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@edsonmedina Could you please make the suggested changes? :) |
Done @xabbuh |
@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. :) |
@xabbuh check it now |
It's perfect now. Thanks @edsonmedina! |
👍 |
Thanks Edson for the PR and working with us with the updates. It's very nice now! |
No description provided.