-
Notifications
You must be signed in to change notification settings - Fork 887
feat: Add vscodeipc
subcommand for VS Code Extension
#5326
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
Conversation
This enables the VS Code extension to communicate with a Coder client. The extension will download the slim binary from `/bin/*` for the respective client architecture and OS, then execute `coder vscodeipc` for the connecting workspace.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code LGTM, I guess that you can push improvements as follow-ups.
Please document the API (/network
, /port
, /execute
) or link it to the extension. I haven't seen the client code, so I don't know if it needs to be versioned (/v1
).
Co-authored-by: Mathias Fredriksson <mafredri@gmail.com>
Co-authored-by: Mathias Fredriksson <mafredri@gmail.com>
Co-authored-by: Mathias Fredriksson <mafredri@gmail.com>
This Pull Request is becoming stale. In order to minimize WIP, prevent merge conflicts and keep the tracker readable, I'm going close to this PR in 3 days if there isn't more activity. |
This enables the VS Code extension to communicate with a Coder client.
The extension will download the slim binary from
/bin/*
for therespective client architecture and OS, then execute
coder vscodeipc
for the connecting workspace.
I don't think I did an amazing job elaborating on why this is required, so if anything is fuzzy in my comments just let me know!