Skip to content

meta: add marco-ippolito to security release stewards #58944

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
Merged
merged 1 commit into from
Jul 9, 2025
Merged

meta: add marco-ippolito to security release stewards #58944

merged 1 commit into from
Jul 9, 2025

Conversation

marco-ippolito
Copy link
Member

@marco-ippolito marco-ippolito commented Jul 3, 2025 •
edited
Loading

I'd like to join the security release stewards

Fixes: nodejs/TSC#1764

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Jul 3, 2025
@marco-ippolito marco-ippolito added the meta Issues and PRs related to the general management of the project. label Jul 3, 2025
@marco-ippolito marco-ippolito requested a review from a team July 3, 2025 14:29
mcollina
mcollina approved these changes Jul 3, 2025
View reviewed changes
Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@richardlau
Copy link
Member

Actually we have a process for this:

node/doc/contributing/security-release-process.md

Lines 11 to 17 in 6a3b545

For each security release, a security steward will take ownership for
coordinating the steps outlined in this process. Security stewards
are nominated through an issue in the TSC repository and approved
through the regular TSC consensus process. Once approved, they
are given access to all of the resources needed to carry out the
steps listed in the process as outlined in
[security steward on/off boarding](security-steward-on-off-boarding.md).

@marco-ippolito
Copy link
Member Author

Actually we have a process for this:

node/doc/contributing/security-release-process.md

Lines 11 to 17 in 6a3b545

For each security release, a security steward will take ownership for
coordinating the steps outlined in this process. Security stewards
are nominated through an issue in the TSC repository and approved
through the regular TSC consensus process. Once approved, they
are given access to all of the resources needed to carry out the
steps listed in the process as outlined in
[security steward on/off boarding](security-steward-on-off-boarding.md).

Oh I couldnt find this. Is it possible to self nominate?

@marco-ippolito marco-ippolito added the blocked PRs that are blocked by other issues or PRs. label Jul 3, 2025
@marco-ippolito
Copy link
Member Author

Opened an issue in the TSC repo nodejs/TSC#1764 for good measure

@marco-ippolito marco-ippolito mentioned this pull request Jul 3, 2025
Closed
@marco-ippolito marco-ippolito removed the blocked PRs that are blocked by other issues or PRs. label Jul 9, 2025
@marco-ippolito
Copy link
Member Author

Approved nodejs/TSC#1764 (comment) merging

@marco-ippolito marco-ippolito added the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 9, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 9, 2025
@nodejs-github-bot nodejs-github-bot merged commit 8b84efc into nodejs:main Jul 9, 2025
29 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in 8b84efc

targos pushed a commit that referenced this pull request Jul 17, 2025
@marco-ippolito @targos
meta: add marco-ippolito to security release stewards
04c7514 
PR-URL: #58944
Fixes: nodejs/TSC#1764
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: Juan José Arboleda <[email protected]>
Reviewed-By: Jordan Harband <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
@github-actions github-actions bot mentioned this pull request Jul 28, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ljharb ljharb ljharb approved these changes

@mcollina mcollina mcollina approved these changes

@jasnell jasnell jasnell approved these changes

@richardlau richardlau richardlau approved these changes

@gireeshpunathil gireeshpunathil gireeshpunathil approved these changes

@legendecas legendecas legendecas approved these changes

@juanarbol juanarbol juanarbol approved these changes

@RafaelGSS RafaelGSS RafaelGSS approved these changes

@RaisinTen RaisinTen RaisinTen approved these changes

Assignees
No one assigned
Labels
doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Self-nominate for security release steward