Skip to content

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

Merged
merged 15 commits into from
Dec 18, 2022
Merged

Conversation

kylecarbs
Copy link
Member

@kylecarbs kylecarbs commented Dec 6, 2022

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.

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!

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.
@kylecarbs kylecarbs self-assigned this Dec 6, 2022
Copy link
Member

@mtojek mtojek left a 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).

kylecarbs and others added 3 commits December 9, 2022 07:22
Co-authored-by: Mathias Fredriksson <mafredri@gmail.com>
Co-authored-by: Mathias Fredriksson <mafredri@gmail.com>
Co-authored-by: Mathias Fredriksson <mafredri@gmail.com>
@github-actions
Copy link

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.

@github-actions github-actions bot added the stale This issue is like stale bread. label Dec 17, 2022
@kylecarbs kylecarbs removed the stale This issue is like stale bread. label Dec 18, 2022
@kylecarbs kylecarbs merged commit e61234f into main Dec 18, 2022
@kylecarbs kylecarbs deleted the vscodeext branch December 18, 2022 23:50
@github-actions github-actions bot locked and limited conversation to collaborators Dec 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants