Skip to content

src: add native debugging code to workers - #21423

Closed
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:worker-debug
Closed

src: add native debugging code to workers#21423
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:worker-debug

Conversation

@addaleax

Copy link
Copy Markdown
Member

Now that we have better native debugging utilities in core,
let’s use them :)

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Now that we have better native debugging utilities in core,
let’s use them :)
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-botnodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Jun 20, 2018
@addaleax

Copy link
Copy Markdown
MemberAuthor

@BridgeARBridgeAR added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jun 20, 2018
@targos

Copy link
Copy Markdown
Member

@apapirovski

Copy link
Copy Markdown
Contributor

Landed in 018d618

apapirovski pushed a commit that referenced this pull request Jun 25, 2018
Now that we have better native debugging utilities in core,
let’s use them :)
PR-URL: #21423
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Matheus Marchini <matheus@sthima.com>
targos pushed a commit that referenced this pull request Jun 25, 2018
Now that we have better native debugging utilities in core,
let’s use them :)
PR-URL: #21423
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Matheus Marchini <matheus@sthima.com>
@targostargos mentioned this pull request Jul 3, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.c++Issues and PRs that require attention from people who are familiar with C++.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants

@addaleax@nodejs-github-bot@targos@apapirovski@jasnell@TimothyGu@cjihrig@mmarchini@devsnek@BridgeAR