Skip to content

report: add more memory info - #45254

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
theanarkh:add_memory_info_for_report
Nov 2, 2022
Merged

report: add more memory info#45254
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
theanarkh:add_memory_info_for_report

Conversation

@theanarkh

Copy link
Copy Markdown
Contributor

add more memory info.

  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

@nodejs-github-botnodejs-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. report Issues and PRs related to process.report. labels Oct 31, 2022
@theanarkh
theanarkhforce-pushed the add_memory_info_for_report branch from 56ddd69 to a757c08CompareOctober 31, 2022 16:00
@theanarkh
theanarkhforce-pushed the add_memory_info_for_report branch from a757c08 to 09bf6caCompareOctober 31, 2022 22:56
@theanarkh
theanarkhforce-pushed the add_memory_info_for_report branch from 09bf6ca to 74d98bfCompareOctober 31, 2022 22:59
@theanarkhtheanarkh added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 2, 2022
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 2, 2022
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@theanarkhtheanarkh added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 2, 2022
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 2, 2022
@nodejs-github-bot
nodejs-github-bot merged commit 070e111 into nodejs:mainNov 2, 2022
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 070e111

RafaelGSS pushed a commit that referenced this pull request Nov 10, 2022
PR-URL: #45254
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
@RafaelGSSRafaelGSS mentioned this pull request Nov 10, 2022
danielleadams pushed a commit that referenced this pull request Dec 30, 2022
PR-URL: #45254
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
danielleadams pushed a commit that referenced this pull request Jan 3, 2023
PR-URL: #45254
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++Issues and PRs that require attention from people who are familiar with C++.needs-ciPRs that need a full CI run.reportIssues and PRs related to process.report.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@theanarkh@nodejs-github-bot@jasnell@gireeshpunathil@legendecas