Skip to content

doc: add importing util to example of `process.report.getReport' - #47298

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
deokjinkim:230330_doc_add_util_process
Mar 31, 2023
Merged

doc: add importing util to example of `process.report.getReport'#47298
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
deokjinkim:230330_doc_add_util_process

Conversation

@deokjinkim

@deokjinkimdeokjinkim commented Mar 29, 2023

Copy link
Copy Markdown
Contributor

util.inspect() is used, but util is not imported.
So added importing util to example of process.report.getReport.
Plus, fix wrong importing in example of process.memoryUsage.rss().

@deokjinkim
deokjinkimforce-pushed the 230330_doc_add_util_process branch from ff05a69 to 6bb123dCompareMarch 29, 2023 15:55
@VoltrexKeyvaVoltrexKeyva added doc Issues and PRs related to the documentations. process Issues and PRs related to the process subsystem. labels Mar 29, 2023
util.inspect() is used, but `util` is not imported. So added
importing util to example of `process.report.getReport`.
Plus, fix wrong importing in example of `process.memoryUsage.rss()`.
@deokjinkim
deokjinkimforce-pushed the 230330_doc_add_util_process branch from 6bb123d to 454c6fcCompareMarch 30, 2023 13:40
@deokjinkimdeokjinkim added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Mar 30, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 31, 2023
@nodejs-github-bot
nodejs-github-bot merged commit accfe8e into nodejs:mainMar 31, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in accfe8e

RafaelGSS pushed a commit that referenced this pull request Apr 5, 2023
util.inspect() is used, but `util` is not imported. So added
importing util to example of `process.report.getReport`.
Plus, fix wrong importing in example of `process.memoryUsage.rss()`.
PR-URL: #47298
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
@RafaelGSSRafaelGSS mentioned this pull request Apr 6, 2023
RafaelGSS pushed a commit that referenced this pull request Apr 6, 2023
util.inspect() is used, but `util` is not imported. So added
importing util to example of `process.report.getReport`.
Plus, fix wrong importing in example of `process.memoryUsage.rss()`.
PR-URL: #47298
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Apr 7, 2023
util.inspect() is used, but `util` is not imported. So added
importing util to example of `process.report.getReport`.
Plus, fix wrong importing in example of `process.memoryUsage.rss()`.
PR-URL: #47298
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Apr 8, 2023
util.inspect() is used, but `util` is not imported. So added
importing util to example of `process.report.getReport`.
Plus, fix wrong importing in example of `process.memoryUsage.rss()`.
PR-URL: #47298
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
danielleadams pushed a commit that referenced this pull request Jul 6, 2023
util.inspect() is used, but `util` is not imported. So added
importing util to example of `process.report.getReport`.
Plus, fix wrong importing in example of `process.memoryUsage.rss()`.
PR-URL: #47298
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
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.docIssues and PRs related to the documentations.processIssues and PRs related to the process subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@deokjinkim@nodejs-github-bot@lpinca@cjihrig@HarshithaKP@VoltrexKeyva