Skip to content

Conversation

@ofrobots
Copy link
Contributor

@ofrobots ofrobots commented Jul 15, 2016

Checklist
  • make -j4 test (UNIX), or vcbuild test nosign (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

deps:v8_inspector

Description of change

Remove wtf files as v8_inspector no longer needs them.

Ref: #7123

Remove wtf files as v8_inspector no longer needs them.

Ref: nodejs#7123
@nodejs-github-bot nodejs-github-bot added the inspector Issues and PRs related to the V8 inspector protocol label Jul 15, 2016
@cjihrig
Copy link
Contributor

cjihrig commented Jul 15, 2016

LGTM

@ofrobots
Copy link
Contributor Author

@targos
Copy link
Member

targos commented Jul 15, 2016

LGTM

1 similar comment
@JungMinu
Copy link
Member

LGTM

@gibfahn gibfahn mentioned this pull request Jul 18, 2016
@targos
Copy link
Member

targos commented Jul 19, 2016

landed in 257a866

@targos targos closed this Jul 19, 2016
targos pushed a commit that referenced this pull request Jul 19, 2016
Remove wtf files as v8_inspector no longer needs them.

Ref: #7123

PR-URL: #7751
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Minwoo Jung <[email protected]>
evanlucas pushed a commit that referenced this pull request Jul 19, 2016
Remove wtf files as v8_inspector no longer needs them.

Ref: #7123

PR-URL: #7751
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Minwoo Jung <[email protected]>
evanlucas pushed a commit that referenced this pull request Jul 20, 2016
Remove wtf files as v8_inspector no longer needs them.

Ref: #7123

PR-URL: #7751
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Minwoo Jung <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

inspector Issues and PRs related to the V8 inspector protocol

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants