Skip to content

Conversation

@kofta999
Copy link
Contributor

Description

Adjusted the formatting of the "RangeError: Maximum call stack size exceeded" message in event-loop-timers-and-nexttick.md. This change ensures that "from v8" is no longer included within the code-style formatting (backticks), improving readability.

Validation

Related Issues

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

Signed-off-by: Mostafa Mahmoud <mostafaxxx555@gmail.com>
@kofta999 kofta999 requested a review from a team as a code owner July 14, 2025 11:33
@vercel
Copy link

vercel bot commented Jul 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nodejs-org ✅ Ready (Inspect) Visit Preview Jul 14, 2025 11:34am

@kofta999 kofta999 changed the title docs: Adjust V8 RangeError message formatting fix(learn): Adjust V8 RangeError message formatting Jul 14, 2025
@codecov
Copy link

codecov bot commented Jul 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.15%. Comparing base (8d8aeae) to head (a2d2a49).
Report is 5 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7977   +/-   ##
=======================================
  Coverage   73.15%   73.15%           
=======================================
  Files          95       95           
  Lines        8355     8355           
  Branches      218      218           
=======================================
  Hits         6112     6112           
  Misses       2242     2242           
  Partials        1        1           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@avivkeller avivkeller enabled auto-merge July 16, 2025 11:08
@avivkeller avivkeller added the github_actions:pull-request Trigger Pull Request Checks label Jul 16, 2025
@github-actions github-actions bot removed the github_actions:pull-request Trigger Pull Request Checks label Jul 16, 2025
@avivkeller avivkeller added this pull request to the merge queue Jul 16, 2025
Merged via the queue into nodejs:main with commit 77f675c Jul 16, 2025
13 checks passed
@kofta999 kofta999 deleted the patch-1 branch July 16, 2025 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants