Skip to content

src: use SPrintF in ProcessEmitWarning - #38758

Closed
RaisinTen wants to merge 3 commits into
nodejs:masterfrom
RaisinTen:src/use-SPrintF-in-ProcessEmitWarning
Closed

src: use SPrintF in ProcessEmitWarning#38758
RaisinTen wants to merge 3 commits into
nodejs:masterfrom
RaisinTen:src/use-SPrintF-in-ProcessEmitWarning

Conversation

@RaisinTen

Copy link
Copy Markdown
Member

No description provided.

@github-actionsgithub-actionsBot added c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. labels May 21, 2021
addaleax
addaleax previously approved these changes May 22, 2021
Comment threadsrc/node_process.h Outdated
@addaleax
addaleax dismissed their stale reviewMay 22, 2021 13:52

code changed

@nodejs-github-bot

This comment has been minimized.

@RaisinTenRaisinTen added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label May 22, 2021
@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

nodejs-github-bot commented May 24, 2021

Copy link
Copy Markdown
Collaborator

@jasnell

Copy link
Copy Markdown
Member

Landed in fa7cdd6

@jasnelljasnell closed this May 25, 2021
jasnell pushed a commit that referenced this pull request May 25, 2021
PR-URL: #38758
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
@RaisinTen
RaisinTen deleted the src/use-SPrintF-in-ProcessEmitWarning branch May 25, 2021 14:17
danielleadams pushed a commit that referenced this pull request May 31, 2021
PR-URL: #38758
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
@danielleadamsdanielleadams mentioned this pull request May 31, 2021
@richardlau

Copy link
Copy Markdown
Member

Doesn't land cleanly on v14.x-staging.

@RaisinTen

Copy link
Copy Markdown
MemberAuthor

@richardlau Backport PR: #39419

richardlau pushed a commit to RaisinTen/node that referenced this pull request Jul 19, 2021
PR-URL: nodejs#38758
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Backport-PR-URL: nodejs#39419
richardlau pushed a commit that referenced this pull request Jul 19, 2021
PR-URL: #38758
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Backport-PR-URL: #39419
richardlau pushed a commit that referenced this pull request Jul 20, 2021
PR-URL: #38758
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Backport-PR-URL: #39419
@richardlaurichardlau mentioned this pull request Jul 20, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.c++Issues and PRs that require attention from people who are familiar with C++.needs-ciPRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@RaisinTen@nodejs-github-bot@jasnell@richardlau@addaleax@joyeecheung@targos