Skip to content

doc: use _Class Method_ in async_hooks.md#34626

Merged
Trott merged 0 commit into
nodejs:masterfrom
Trott:class-method
Aug 7, 2020
Merged

doc: use _Class Method_ in async_hooks.md#34626
Trott merged 0 commit into
nodejs:masterfrom
Trott:class-method

Conversation

@Trott

@Trott Trott commented Aug 4, 2020

Copy link
Copy Markdown
Member

We use Class Method rather than static in documentation headers in
buffer.md and crypto.md. We use static in one place in
async_hooks.md. Change to Class Method for consistency.

Checklist

@nodejs-github-bot

nodejs-github-bot commented Aug 4, 2020

Copy link
Copy Markdown
Collaborator

Lite-CI: https://ci.nodejs.org/job/node-test-pull-request-lite-pipeline/4206

@nodejs-github-bot nodejs-github-bot added async_hooks Issues and PRs related to the async hooks subsystem. doc Issues and PRs related to the documentations. labels Aug 4, 2020
@Trott Trott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Aug 5, 2020
@Trott Trott closed this Aug 7, 2020
@Trott Trott merged commit e692adf into nodejs:master Aug 7, 2020
@Trott

Trott commented Aug 7, 2020

Copy link
Copy Markdown
Member Author

Landed in e692adf

@Trott Trott deleted the class-method branch August 7, 2020 01:17
addaleax pushed a commit that referenced this pull request Aug 8, 2020
@Trott @addaleax
doc: use _Class Method_ in async_hooks.md
b78278b 
We use _Class Method_ rather than _static_ in documentation headers in
buffer.md and crypto.md.  We use _static_ in one place in
async_hooks.md. Change to _Class Method_ for consistency.

PR-URL: #34626
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
@codebytere codebytere mentioned this pull request Aug 10, 2020
Merged
codebytere pushed a commit that referenced this pull request Aug 11, 2020
@Trott @codebytere
doc: use _Class Method_ in async_hooks.md
a944dab 
We use _Class Method_ rather than _static_ in documentation headers in
buffer.md and crypto.md.  We use _static_ in one place in
async_hooks.md. Change to _Class Method_ for consistency.

PR-URL: #34626
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
addaleax pushed a commit that referenced this pull request Sep 22, 2020
@Trott @addaleax
doc: use _Class Method_ in async_hooks.md
5846bef 
We use _Class Method_ rather than _static_ in documentation headers in
buffer.md and crypto.md.  We use _static_ in one place in
async_hooks.md. Change to _Class Method_ for consistency.

PR-URL: #34626
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
@codebytere codebytere mentioned this pull request Sep 28, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jasnell jasnell jasnell approved these changes

@addaleax addaleax addaleax approved these changes

@joyeecheung joyeecheung joyeecheung approved these changes

@Flarna Flarna Flarna approved these changes

Assignees

No one assigned

Labels

async_hooks Issues and PRs related to the async hooks subsystem. 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.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@Trott @nodejs-github-bot @jasnell @addaleax @joyeecheung @Flarna