Skip to content

doc: update console.error example to API doc - #34964

Closed
iyabong wants to merge 3 commits into
nodejs:masterfrom
iyabong:master_add_example_console_md
Closed

doc: update console.error example to API doc#34964
iyabong wants to merge 3 commits into
nodejs:masterfrom
iyabong:master_add_example_console_md

Conversation

@iyabong

Copy link
Copy Markdown
Contributor

'console.error(new Error())' method
prints not only custom message but also stack trace.

So i added stack trace lines to example box.

Signed-off-by: Lee, Bonggi iyabong@gmail.com

Checklist

'console.error(new Error())' method
prints not only custom message but also stack trace.
So i added stack trace lines to example box.
Signed-off-by: Lee, Bonggi <iyabong@gmail.com>
@nodejs-github-botnodejs-github-bot added console Issues and PRs related to the console subsystem. doc Issues and PRs related to the documentations. labels Aug 29, 2020

@aduh95aduh95 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello 👋 Thanks for your work, that seems like a neat suggestion. Just a few comments:

Comment threaddoc/api/console.md Outdated
Comment threaddoc/api/console.md Outdated
iyabongand others added 2 commits October 24, 2020 14:30
remove OS specific console log
Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
remove custom message
Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
@aduh95aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. review wanted PRs that need reviews. labels Oct 24, 2020
aduh95 pushed a commit that referenced this pull request Oct 24, 2020
Signed-off-by: Lee, Bonggi <iyabong@gmail.com>
PR-URL: #34964
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
@aduh95

aduh95 commented Oct 24, 2020

Copy link
Copy Markdown
Contributor

Landed in 440edaa, thanks again and congrats for your first commit in Node.js @iyabong!

@aduh95aduh95 closed this Oct 24, 2020
targos pushed a commit that referenced this pull request Nov 3, 2020
Signed-off-by: Lee, Bonggi <iyabong@gmail.com>
PR-URL: #34964
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
@targostargos mentioned this pull request Nov 3, 2020
BethGriggs pushed a commit that referenced this pull request Dec 8, 2020
Signed-off-by: Lee, Bonggi <iyabong@gmail.com>
PR-URL: #34964
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
BethGriggs pushed a commit that referenced this pull request Dec 10, 2020
Signed-off-by: Lee, Bonggi <iyabong@gmail.com>
PR-URL: #34964
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
@BethGriggsBethGriggs mentioned this pull request Dec 10, 2020
BethGriggs pushed a commit that referenced this pull request Dec 15, 2020
Signed-off-by: Lee, Bonggi <iyabong@gmail.com>
PR-URL: #34964
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.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.consoleIssues and PRs related to the console subsystem.docIssues and PRs related to the documentations.review wantedPRs that need reviews.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@iyabong@aduh95@gireeshpunathil@nodejs-github-bot