Skip to content

Conversation

@efekrskl
Copy link
Contributor

@efekrskl efekrskl commented Nov 4, 2025

Description

I first just wanted to replace the outdated Microsoft Edge link with the correct DevTools doc, but it looks like the debugging section overall is out of date. This PR aims to improve the docs a bit (at least partially)

Would it make sense to expand this into a broader rework later?

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.

@efekrskl efekrskl requested a review from a team as a code owner November 4, 2025 21:37
Copilot AI review requested due to automatic review settings November 4, 2025 21:37
@vercel
Copy link

vercel bot commented Nov 4, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
nodejs-org Ready Ready Preview Nov 5, 2025 4:04pm

Copy link
Contributor

Copilot AI left a 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 updates the debugging documentation to improve clarity and organization of the DevTools connection instructions, and modernizes several external reference links.

  • Restructured Chrome/Edge DevTools instructions into two distinct options with clearer step-by-step guidance
  • Updated external documentation links to more specific and current resources
  • Removed the Gitpod section from the debugging tools list

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mikeesto
Copy link
Member

mikeesto commented Nov 6, 2025

LGTM, thanks

@avivkeller avivkeller enabled auto-merge November 6, 2025 15:35
@codecov
Copy link

codecov bot commented Nov 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.67%. Comparing base (9fbb9f6) to head (2044273).
⚠️ Report is 4 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8329      +/-   ##
==========================================
- Coverage   76.72%   76.67%   -0.06%     
==========================================
  Files         118      118              
  Lines        9805     9805              
  Branches      335      336       +1     
==========================================
- Hits         7523     7518       -5     
- Misses       2280     2285       +5     
  Partials        2        2              

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

@avivkeller avivkeller added this pull request to the merge queue Nov 6, 2025
Merged via the queue into nodejs:main with commit e2b6839 Nov 6, 2025
12 checks passed
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.

3 participants