-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Enforce the "one sentence per line" rule in docs #4377
Conversation
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.
I made a few comments below.
👍 for the "one sentence per line" rule! The single commit didn't ease the review though.
This pull request has been automatically marked as stale because it has not had recent activity, and will be closed if no further activity occurs. If this pull request was overlooked, forgotten, or should remain open for any other reason, please reply here to call attention to it and remove the stale status. Thank you for your contributions. |
@pchaigno Anything left to do here? |
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.
This is going to cause conflicts on one of my PRs too, but that'll teach me for leaving it so long to review. 😆
To make diffs easier to review, I've refactored
README.md
andCONTRIBUTING.md
to limit lines to no more than one sentence.In addition, I fixed a few typos, made more optimal use of reference links, and made shell-commands tagged code-blocks to enable highlighting.
Refs: #4271 (comment)
/cc @pchaigno because this will probably break his pull-request