Skip to content

inspector: reduce InspectorIo API surface - #28526

Closed
eugeneo wants to merge 1 commit into
nodejs:masterfrom
eugeneo:pipe-io
Closed

inspector: reduce InspectorIo API surface#28526
eugeneo wants to merge 1 commit into
nodejs:masterfrom
eugeneo:pipe-io

Conversation

@eugeneo

Copy link
Copy Markdown
Contributor

This is a cleanup, allowing for a better separation of concerns.

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

@nodejs-github-botnodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. lib / src Issues and PRs related to general changes in the lib or src directory. labels Jul 3, 2019
@eugeneo
eugeneo requested a review from alexkozyJuly 3, 2019 18:13
@eugeneoeugeneo added the inspector Issues and PRs related to the V8 inspector protocol label Jul 3, 2019
This is a cleanup, allowing for a better separation of concerns.
@BridgeARBridgeAR added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jul 4, 2019
@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@addaleax

Copy link
Copy Markdown
Member

Landed in 17862fc

@addaleaxaddaleax closed this Jul 6, 2019
addaleax pushed a commit that referenced this pull request Jul 6, 2019
This is a cleanup, allowing for a better separation of concerns.
PR-URL: #28526
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Aleksei Koziatinskii <ak239spb@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
targos pushed a commit that referenced this pull request Jul 20, 2019
This is a cleanup, allowing for a better separation of concerns.
PR-URL: #28526
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Aleksei Koziatinskii <ak239spb@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
This was referenced Jul 23, 2019
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++.inspectorIssues and PRs related to the V8 inspector protocollib / srcIssues and PRs related to general changes in the lib or src directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@eugeneo@nodejs-github-bot@addaleax@alexkozy@Trott@BridgeAR