Skip to content

Update built-in debugger link to latest#1913

Merged
fhemberger merged 2 commits into
masterfrom
latest-debugger
Dec 19, 2018
Merged

Update built-in debugger link to latest#1913
fhemberger merged 2 commits into
masterfrom
latest-debugger

Conversation

@ZYSzys

@ZYSzys ZYSzys commented Nov 26, 2018

Copy link
Copy Markdown
Member

Update built-in debugger link to the latest in guides/debugging-getting-started.

Comment thread locale/en/docs/guides/debugging-getting-started.md
@refack refack added the content Issues/pr concerning content label Nov 26, 2018
Comment thread locale/en/docs/guides/debugging-getting-started.md
@ZYSzys

ZYSzys commented Nov 27, 2018

Copy link
Copy Markdown
Member Author

Find that it's indeed the Legacy Debugger section, I'm kind of confused now whether we should change this.

@fhemberger fhemberger 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.

@ZYSzys Please revert the changes, as it's indeed the legacy debugger.

I only noticed, that in the Korean version, links to the "Built-in Debugger" point to

So these links should be replaced with https://nodejs.org/dist/latest-v6.x/docs/api/debugger.html, as in the other languages.

If I'm not mistaken, this should be the only remaining change.

@ZYSzys

ZYSzys commented Dec 19, 2018

Copy link
Copy Markdown
Member Author

@fhemberger 👌Reverted and updated the link of "Built-in Debugger" in the Korean version.

@fhemberger fhemberger merged commit d001e32 into master Dec 19, 2018
@fhemberger fhemberger deleted the latest-debugger branch December 19, 2018 10:58
@fhemberger

Copy link
Copy Markdown
Contributor

@ZYSzys Thanks! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content Issues/pr concerning content

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants