Skip to content
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

backport: Deprecate old debug protocol in docs #11172

Closed
wants to merge 1 commit into from

Conversation

jkrems
Copy link
Contributor

@jkrems jkrems commented Feb 4, 2017

Original PR: #10320, the only conflict was the V8 protocol URL that changed in the meantime.

Due to changes in V8, the debug protocol will no longer work in Node.js
8.0.0. Note this in the documentation.

PR-URL: #10320
Reviewed-By: Anna Henningsen anna@addaleax.net
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Michael Dawson michael_dawson@ca.ibm.com

Checklist
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

doc

@nodejs-github-bot nodejs-github-bot added debugger doc Issues and PRs related to the documentations. labels Feb 4, 2017
@jkrems jkrems mentioned this pull request Feb 4, 2017
2 tasks
@jkrems jkrems changed the title Backport #10320 to 7.x backport: deprecate debug protocol in docs Feb 4, 2017
@jkrems jkrems changed the title backport: deprecate debug protocol in docs backport: Deprecate old debug protocol in docs Feb 4, 2017
@italoacasas
Copy link
Contributor

@jkrems can you rebase this please.

@jkrems jkrems force-pushed the jk-backport-pr10320 branch 2 times, most recently from f2f0f00 to 6bed46e Compare February 18, 2017 18:24
@jkrems
Copy link
Contributor Author

jkrems commented Feb 18, 2017

Re-applied the commit to the new 7.x-staging branch.

Due to changes in V8, the debug protocol will no longer work in Node.js
8.0.0. Note this in the documentation.

PR-URL: nodejs#10320
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
@jkrems
Copy link
Contributor Author

jkrems commented Mar 20, 2017

Re-applied the commit to the new 7.x-staging branch.

@italoacasas
Copy link
Contributor

Landed in staging

@jkrems jkrems deleted the jk-backport-pr10320 branch March 21, 2017 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants