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.

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

Rephrasing process.initgroups() API docs property in process.md #35611

Merged
merged 1 commit into from
Oct 14, 2020
Merged

Rephrasing process.initgroups() API docs property in process.md #35611

merged 1 commit into from
Oct 14, 2020

Conversation

PoojaDurgad
Copy link
Contributor

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. process Issues and PRs related to the process subsystem. labels Oct 12, 2020
@PoojaDurgad @Trott
doc: use test username instead of real
92610f3 
PR-URL: nodejs#35611
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
@Trott Trott merged commit 92610f3 into nodejs:master Oct 14, 2020
@Trott
Copy link
Member

Trott commented Oct 14, 2020

Landed in 92610f3

MylesBorins pushed a commit that referenced this pull request Oct 14, 2020
@PoojaDurgad @MylesBorins
doc: use test username instead of real
6284f0d 
PR-URL: #35611
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
@MylesBorins MylesBorins mentioned this pull request Oct 14, 2020
Merged
MylesBorins pushed a commit that referenced this pull request Nov 3, 2020
@PoojaDurgad @MylesBorins
doc: use test username instead of real
0071cc5 
PR-URL: #35611
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
@MylesBorins MylesBorins mentioned this pull request Nov 3, 2020
Merged
MylesBorins pushed a commit that referenced this pull request Nov 16, 2020
@PoojaDurgad @MylesBorins
doc: use test username instead of real
42a587f 
PR-URL: #35611
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
@PoojaDurgad @joesepi
doc: use test username instead of real
0de8234 
PR-URL: nodejs#35611
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HarshithaKP HarshithaKP HarshithaKP approved these changes

@addaleax addaleax addaleax approved these changes

@Trott Trott Trott approved these changes

Assignees
No one assigned
Labels
doc Issues and PRs related to the documentations. process Issues and PRs related to the process subsystem.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@PoojaDurgad @Trott @addaleax @HarshithaKP @nodejs-github-bot