-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
docs: Clarify that console.error appears in console but uses stderr #7913
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: Clarify that console.error appears in console but uses stderr #7913
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR improves the documentation for console.error by clarifying that, although error messages are sent to stderr, they still appear in the console.
- Clarifies the behavior of console.error output
- Updates the existing misleading description in the documentation
Comments suppressed due to low confidence (1)
apps/site/pages/en/learn/command-line/output-to-the-command-line-using-nodejs.md:168
- The clarification is an improvement; however, consider explicitly mentioning that error messages are printed via stderr to remove any potential ambiguity.
It will appear in the console, but can be handled separately from regular output.
mikeesto
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks
AugustinMauroy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGMT ! Thanks for your first contribution
|
Lighthouse Results
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #7913 +/- ##
==========================================
- Coverage 75.42% 75.42% -0.01%
==========================================
Files 96 96
Lines 8355 8354 -1
Branches 219 219
==========================================
- Hits 6302 6301 -1
Misses 2051 2051
Partials 2 2 ☔ View full report in Codecov by Sentry. |
Description
Clarified misleading description of
console.errorbehavior in the stdout/stderr documentation. The previous text incorrectly stated thatconsole.error"will not appear in the console" when it actually does appear in the console but uses the stderr stream.Validation
The change is a simple text clarification in documentation. Reviewers should verify that the new wording accurately describes
console.errorbehavior - that it appears in the console like normal output but uses stderr instead of stdout.Related Issues
None - this is a standalone documentation improvement.
Check List
pnpm formatto ensure the code follows the style guide.pnpm testto check if all tests are passing.pnpm buildto check if the website builds without errors.