Skip to content

Remove IPC #182

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
Jul 16, 2018
Merged

Conversation

ivanovit
Copy link
Contributor

@ivanovit ivanovit commented Jun 1, 2018

On windows, IPC connection sometimes fails to be established. Use the debug adapter events for communication with the extension host process.

For more information:
https://code.visualstudio.com/docs/extensionAPI/api-debugging#_debug-session-api
microsoft/vscode#49389

ivanovit added 2 commits June 1, 2018 13:48
Use the debug adapter events
Also added extension host logger
@ivanovit ivanovit self-assigned this Jun 1, 2018
ivanovit added 6 commits June 1, 2018 15:37
The deleted tests were focused on debug protocol. Now these scenarios should be covered by the chrome-debug-adapter-core. Currently, the new tests cover only NS related code.
Current tests for paths are generated for windows paths.
ivanovit added 6 commits July 9, 2018 16:06
In the newer versions of CLI each command can print additional warning before the actual command output.
@ivanovit ivanovit merged commit f4410b4 into iiivanov/use-chrome-debug-adapter-2 Jul 16, 2018
@ivanovit ivanovit deleted the iiivanov/remove-ipc-2 branch July 16, 2018 08:37
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.

3 participants