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

Enforce the "one sentence per line" rule in docs #4377

Merged
merged 5 commits into from
Apr 4, 2019
Merged

Conversation

Alhadis
Copy link
Collaborator

@Alhadis Alhadis commented Jan 11, 2019

To make diffs easier to review, I've refactored README.md and CONTRIBUTING.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

CONTRIBUTING.md Outdated Show resolved Hide resolved
Copy link
Contributor

@pchaigno pchaigno left a 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.

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@stale
Copy link

stale bot commented Mar 22, 2019

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.

@stale stale bot added the Stale label Mar 22, 2019
@Alhadis
Copy link
Collaborator Author

Alhadis commented Mar 22, 2019

@pchaigno Anything left to do here?

@stale stale bot removed the Stale label Mar 22, 2019
@pchaigno pchaigno requested a review from lildude April 4, 2019 12:03
Copy link
Member

@lildude lildude left a 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. 😆

@lildude lildude merged commit f546d83 into master Apr 4, 2019
@lildude lildude deleted the line-breaks branch April 4, 2019 12:16
@github-linguist github-linguist locked as resolved and limited conversation to collaborators Jun 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants