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

doc: add missed `inspect` with numericSeparator to example by deokjinkim · Pull Request #49717 · nodejs/node · GitHub

/ node Public

doc: add missed inspect with numericSeparator to example - #49717

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
deokjinkim:230919_add_missed_inspect_with_numericSeparator
Sep 21, 2023
Merged

doc: add missed inspect with numericSeparator to example#49717
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
deokjinkim:230919_add_missed_inspect_with_numericSeparator

Conversation

Copy link
Copy Markdown
Contributor

In exmple of util.inspect with numericSeparator option, calling util.inspect is missed. So actual result is different from expected result.

Expected result:

1_000 1_000_000 123_456_789n 1_234.123_45 

Actual result:

1000 1000000 123456789n 1234.12345

In exmple of `util.inspect` with numericSeparator option,
calling `util.inspect` is missed. So actual result is different
from expected result.
nodejs-github-bot added doc Issues and PRs related to the documentations. util Issues and PRs related to the built-in util module. labels Sep 19, 2023
MoLow added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Sep 19, 2023
deokjinkim added the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 20, 2023
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 21, 2023
nodejs-github-bot merged commit 4858826 into nodejs:main Sep 21, 2023

Copy link
Copy Markdown
Collaborator

Landed in 4858826

ruyadorno pushed a commit that referenced this pull request Sep 28, 2023
In exmple of `util.inspect` with numericSeparator option,
calling `util.inspect` is missed. So actual result is different
from expected result.

PR-URL: #49717
Reviewed-By: LiviaMedeiros <livia@cirno.name>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
This was referenced Sep 28, 2023
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. doc Issues and PRs related to the documentations. util Issues and PRs related to the built-in util module.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants


Back | FazBrowse Home | New Git URL