FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

debugger: fix after 0450ce7 by indutny · Pull Request #1607 · nodejs/node · GitHub

/ node Public

debugger: fix after 0450ce7 - #1607

Closed
indutny wants to merge 1 commit into
nodejs:masterfrom
indutny:fix/debugger-repl
Closed

debugger: fix after 0450ce7#1607
indutny wants to merge 1 commit into
nodejs:masterfrom
indutny:fix/debugger-repl

Conversation

indutny commented May 4, 2015

Copy link
Copy Markdown
Member

The repl module is now split into internal and public-facing parts.
The debugger is actually using a public facing one, so do it explicitly.

R=@chrisdickinson

The `repl` module is now split into internal and public-facing parts.
The debugger is actually using a public facing one, so do it explicitly.

cjihrig commented May 4, 2015

Copy link
Copy Markdown
Contributor

I haven't tested with the new REPL changes, but does this cause a regression of #49?

indutny commented May 4, 2015

Copy link
Copy Markdown
Member Author

No tests are failing for me.

indutny commented May 4, 2015

Copy link
Copy Markdown
Member Author

Ok, it is crashing, but there is no regression test for it.

mscdex added the debugger label May 4, 2015

Copy link
Copy Markdown
Contributor

Handling this in #1605.

indutny commented May 4, 2015

Copy link
Copy Markdown
Member Author

Ok, closing it then.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

4 participants


Back | FazBrowse Home | New Git URL