FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

src: add fflush() to SnapshotData::ToFile() by addaleax · Pull Request #46531 · nodejs/node · GitHub

/ node Public

src: add fflush() to SnapshotData::ToFile() - #46531

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
addaleax:fflush-in-snapshot
Feb 8, 2023
Merged

src: add fflush() to SnapshotData::ToFile()#46531
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
addaleax:fflush-in-snapshot

Conversation

addaleax commented Feb 6, 2023

Copy link
Copy Markdown
Member

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/startup

nodejs-github-bot 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 Feb 6, 2023
addaleax added the request-ci Add this label to start a Jenkins CI on a PR. label Feb 6, 2023
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Feb 6, 2023

Copy link
Copy Markdown
Collaborator

addaleax added author ready PRs that have at least one approval, no outstanding review comments, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Feb 7, 2023
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Feb 8, 2023
nodejs-github-bot merged commit 5092346 into nodejs:main Feb 8, 2023

Copy link
Copy Markdown
Collaborator

Landed in 5092346

MylesBorins pushed a commit that referenced this pull request Feb 18, 2023
Refs: #46491 (comment)
PR-URL: #46531
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
MylesBorins mentioned this pull request Feb 19, 2023
danielleadams pushed a commit that referenced this pull request Apr 11, 2023
Refs: #46491 (comment)
PR-URL: #46531
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no outstanding review comments, and a CI started. c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants


Back | FazBrowse Home | New Git URL