Skip to content

doc: edit stability text for clarity and style #11112

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

doc: edit stability text for clarity and style #11112

wants to merge 1 commit into from

Conversation

Trott
Copy link
Member

@Trott Trott commented Feb 2, 2017

Minor clarification/style fix to doc on the Locked stability entry.

Checklist
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

doc

@Trott Trott added the doc Issues and PRs related to the documentations. label Feb 2, 2017
@Trott Trott mentioned this pull request Feb 2, 2017
Merged
claudiorodriguez
claudiorodriguez approved these changes Feb 2, 2017
View reviewed changes
Copy link
Contributor

@claudiorodriguez claudiorodriguez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

addaleax
addaleax approved these changes Feb 3, 2017
View reviewed changes
Copy link
Member

@addaleax addaleax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

This probably doesn’t need to wait 72 hours

jasnell pushed a commit that referenced this pull request Feb 3, 2017
@Trott @jasnell
doc: edit stability text for clarity and style
3d7ada3 
PR-URL: #11112
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: Italo A. Casas <[email protected]>
Reviewed-By: Claudio Rodriguez <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
@jasnell
Copy link
Member

jasnell commented Feb 3, 2017

Landed in 3d7ada3

@jasnell jasnell closed this Feb 3, 2017
italoacasas pushed a commit that referenced this pull request Feb 4, 2017
@Trott
doc: edit stability text for clarity and style
2f667d5 
PR-URL: #11112
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: Italo A. Casas <[email protected]>
Reviewed-By: Claudio Rodriguez <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
@italoacasas italoacasas mentioned this pull request Feb 5, 2017
Merged
italoacasas pushed a commit to italoacasas/node that referenced this pull request Feb 14, 2017
@Trott
doc: edit stability text for clarity and style
3ebe306 
PR-URL: nodejs#11112
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: Italo A. Casas <[email protected]>
Reviewed-By: Claudio Rodriguez <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
krydos pushed a commit to krydos/node that referenced this pull request Feb 25, 2017
@Trott @krydos
doc: edit stability text for clarity and style
20c2c8e 
PR-URL: nodejs#11112
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: Italo A. Casas <[email protected]>
Reviewed-By: Claudio Rodriguez <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
jasnell pushed a commit that referenced this pull request Mar 7, 2017
@Trott @jasnell
doc: edit stability text for clarity and style
f717b3b 
PR-URL: #11112
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: Italo A. Casas <[email protected]>
Reviewed-By: Claudio Rodriguez <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
jasnell pushed a commit that referenced this pull request Mar 7, 2017
@Trott @jasnell
doc: edit stability text for clarity and style
6f81f99 
PR-URL: #11112
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: Italo A. Casas <[email protected]>
Reviewed-By: Claudio Rodriguez <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
MylesBorins pushed a commit that referenced this pull request Mar 9, 2017
@Trott @MylesBorins
doc: edit stability text for clarity and style
d404d8b 
PR-URL: #11112
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: Italo A. Casas <[email protected]>
Reviewed-By: Claudio Rodriguez <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
@MylesBorins MylesBorins mentioned this pull request Mar 9, 2017
Merged
MylesBorins pushed a commit that referenced this pull request Mar 9, 2017
@Trott @MylesBorins
doc: edit stability text for clarity and style
c2d7090 
PR-URL: #11112
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: Italo A. Casas <[email protected]>
Reviewed-By: Claudio Rodriguez <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
@MylesBorins MylesBorins mentioned this pull request Mar 9, 2017
Merged
@Trott Trott deleted the locked-locked branch January 13, 2022 22:42
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

+4 more reviewers

@thefourtheye thefourtheye thefourtheye approved these changes

@TimothyGu TimothyGu TimothyGu approved these changes

@claudiorodriguez claudiorodriguez claudiorodriguez approved these changes

@italoacasas italoacasas italoacasas approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

doc Issues and PRs related to the documentations.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@Trott @jasnell @thefourtheye @addaleax @TimothyGu @claudiorodriguez @joyeecheung @italoacasas