Skip to content

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

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

Copy link
Copy Markdown
Contributor

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

@indutny

Copy link
Copy Markdown
MemberAuthor

No tests are failing for me.

@indutny

Copy link
Copy Markdown
MemberAuthor

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

@chrisdickinson

Copy link
Copy Markdown
Contributor

Handling this in #1605.

@indutny

Copy link
Copy Markdown
MemberAuthor

Ok, closing it then.

Sign up for freeto 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

@indutny@cjihrig@chrisdickinson@mscdex