Skip to content

Conversation

@RafaelGSS
Copy link
Member

This should solve nodejs/security-wg#1453 (comment)

cc: @nodejs/security-wg

@RafaelGSSRafaelGSS requested a review from richardlauMay 9, 2025 22:42
@nodejs-github-botnodejs-github-bot added the meta Issues and PRs related to the general management of the project. label May 9, 2025
@aduh95
Copy link
Contributor

It'd be less confusing to amend the commit message to say it's CodeQL related

Copy link
Member

@bjohansebasbjohansebas left a comment

Choose a reason for hiding this comment

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

LGTM

@aduh95aduh95 merged commit 0c78a48 into nodejs:mainMay 15, 2025
17 checks passed
@aduh95
Copy link
Contributor

Landed in 0c78a48

targos pushed a commit that referenced this pull request May 16, 2025
PR-URL: #58254 Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
aduh95 pushed a commit that referenced this pull request Jun 10, 2025
PR-URL: #58254 Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

@RafaelGSS@aduh95@jasnell@benjamingr@lpinca@bjohansebas@nodejs-github-bot