Skip to content

tools: add security-wg as dep updaters owner - #48113

Merged
nodejs-github-bot merged 3 commits into
nodejs:mainfrom
marco-ippolito:feat/security-wg-codeowners
May 24, 2023
Merged

tools: add security-wg as dep updaters owner#48113
nodejs-github-bot merged 3 commits into
nodejs:mainfrom
marco-ippolito:feat/security-wg-codeowners

Conversation

@marco-ippolito

@marco-ippolitomarco-ippolito commented May 22, 2023

Copy link
Copy Markdown
Member

Refs: nodejs/security-wg#828
Added security-wg as codeowner of dep updaters

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/tsc

@nodejs-github-botnodejs-github-bot added the meta Issues and PRs related to the general management of the project. label May 22, 2023
Comment thread.github/CODEOWNERS Outdated

@RafaelGSSRafaelGSS left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM with @richardlau suggestion

Comment thread.github/CODEOWNERS Outdated
Co-authored-by: Rich Trott <rtrott@gmail.com>
@TrottTrott added the commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. label May 23, 2023
@TrottTrott added the commit-queue Add this label to land a pull request using GitHub Actions. label May 23, 2023
@marco-ippolitomarco-ippolito added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels May 24, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label May 24, 2023
@nodejs-github-bot
nodejs-github-bot merged commit bcf1dd0 into nodejs:mainMay 24, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in bcf1dd0

targos pushed a commit that referenced this pull request May 30, 2023
PR-URL: #48113
Refs: nodejs/security-wg#828
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@targostargos mentioned this pull request Jun 4, 2023
danielleadams pushed a commit that referenced this pull request Jul 6, 2023
PR-URL: #48113
Refs: nodejs/security-wg#828
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
MoLow pushed a commit to MoLow/node that referenced this pull request Jul 6, 2023
PR-URL: nodejs#48113
Refs: nodejs/security-wg#828
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
PR-URL: nodejs#48113
Refs: nodejs/security-wg#828
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
PR-URL: nodejs#48113
Refs: nodejs/security-wg#828
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commit-queue-squashAdd this label to instruct the Commit Queue to squash all the PR commits into the first one.metaIssues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@marco-ippolito@nodejs-github-bot@Trott@lpinca@richardlau@RafaelGSS@RaisinTen