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

doc: standardize on pseudorandom #38196

Closed
wants to merge 3 commits into from
Closed

Conversation

Trott
Copy link
Member

@Trott Trott commented Apr 11, 2021

Our docs use both pseudo-random and pseudorandom. Standardize on pseudorandom.

Which spelling to use is not covered in our doc style guide. It says to refer to Microsoft's style guide for things not covered. Microsoft's style guide similarly does not indicate which spelling to use. It refers to the American Heritage Dictionary for things it doesn't cover. American Heritage Dictionary contains an entry for pseudorandom and does not appear to mention pseudo-random. (Anecdotally, pseudorandom seems to be the most common spelling in recent documents and _pseudo-random tends to show up in older documents.)

Our docs use both _pseudo-random_ and _pseudorandom_. Standardize on
_pseudorandom_.
Our docs use both _pseudo-random_ and _pseudorandom_. Standardize on
_pseudorandom_.
Our docs use both _pseudo-random_ and _pseudorandom_. Standardize on
_pseudorandom_.
@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Apr 11, 2021
@RaisinTen RaisinTen added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 11, 2021
@jasnell jasnell added the fast-track PRs that do not need to wait for 48 hours to land. label Apr 12, 2021
@jasnell
Copy link
Member

jasnell commented Apr 12, 2021

There's really no reason to make this one wait the full 48 hours. Please 👍🏻 to fast-track?

RaisinTen pushed a commit that referenced this pull request Apr 13, 2021
Our docs use both _pseudo-random_ and _pseudorandom_. Standardize on
_pseudorandom_.

PR-URL: #38196
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@RaisinTen
Copy link
Contributor

Landed in 4def7c4

@RaisinTen RaisinTen closed this Apr 13, 2021
BethGriggs pushed a commit that referenced this pull request Apr 15, 2021
Our docs use both _pseudo-random_ and _pseudorandom_. Standardize on
_pseudorandom_.

PR-URL: #38196
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@Trott Trott deleted the psuedorandom branch September 25, 2022 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 48 hours to land.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants