Skip to content

Conversation

@vsemozhetbyt
Copy link
Contributor

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

Backport of #19743 (proactive)

Commit 1: replace 2 hyphens (--) by spaced m-dash () as per STYLE_GUIDE.md.
Commit 2: space infix operators.
Commit 3: unify quotes in inline code spans (use only single quotes).
Commit 4: unify * Returns: (eliminate deviations).
Commit 5: dedupe spaces.

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. v6.x labels Apr 2, 2018
@vsemozhetbyt vsemozhetbyt mentioned this pull request Apr 2, 2018
3 tasks
@vsemozhetbyt
Copy link
Contributor Author

Refs: #19743 (review)

@vsemozhetbyt
Copy link
Contributor Author

* Replace 2 hyphens (--) by spaced m-dashes (—) as per STYLE_GUIDE.md.
* Space infix operators.
* Unify quotes in inline code spans (use only single quotes).
* Unify `* Returns:` (eliminate deviations).
* Dedupe spaces.

PR-URL: #19743
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
@vsemozhetbyt
Copy link
Contributor Author

I've copied the landed PR metadata in the first commit and reword titles in the other commits to hint fixup.

CI-lite: https://ci.nodejs.org/job/node-test-pull-request-lite/419/

MylesBorins pushed a commit that referenced this pull request Apr 13, 2018
* Replace 2 hyphens (--) by spaced m-dashes (—) as per STYLE_GUIDE.md.
* Space infix operators.
* Unify quotes in inline code spans (use only single quotes).
* Unify `* Returns:` (eliminate deviations).
* Dedupe spaces.

Backport-PR-URL: #19761
PR-URL: #19743
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
@MylesBorins
Copy link
Contributor

landed in 4d67369

@vsemozhetbyt vsemozhetbyt deleted the backport-19743-to-v6.x branch April 13, 2018 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants