Skip to content

Fix broken link in nodejs-debugging.md #8451

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

Merged
merged 1 commit into from
Jun 6, 2025

Conversation

alefteris
Copy link
Contributor

No description provided.

@Copilot Copilot AI review requested due to automatic review settings June 6, 2025 07:19
Copy link
Contributor

@Copilot 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 fixes a broken link in the Node.js debugging documentation by updating the URL reference for the runtimeExecutable attribute.

  • Updated the link for the runtimeExecutable configuration from an incorrect file to the correct docs file.
  • Ensured consistency in documentation anchors for Node.js debugging settings.
Comments suppressed due to low confidence (1)

docs/nodejs/nodejs-debugging.md:122

  • Verify that the anchor 'launch-configuration-support-for-npm-and-other-tools' exists in the nodejs-debugging.md file to ensure the link works as expected.
* `runtimeExecutable` - absolute path to the runtime executable to be used. Default is `node`. See section [Launch configuration support for 'npm' and other tools](/docs/nodejs/nodejs-debugging.md#launch-configuration-support-for-npm-and-other-tools).

Copy link

@DhruvTheDev1 DhruvTheDev1 left a comment

Choose a reason for hiding this comment

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

Nice fix! Confirmed that the updated link correctly points to the right section.

Copy link
Contributor

@ntrogh ntrogh left a comment

Choose a reason for hiding this comment

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

@alefteris Thanks for fixing the broken link!

@ntrogh ntrogh merged commit 4d1c6aa into microsoft:main Jun 6, 2025
1 of 2 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