Skip to content
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

Add support for VSCode based IDE inside Flatpak #1577

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

acolombier
Copy link
Contributor

This adds support for VSCode based IDE installed via Flatpak on Linux. Note that this depends on a limitation in open-remote-ssh

Note: this is also adding support for Codium Insiders. Let me know if you want me to split the commit or PR

@acolombier acolombier force-pushed the feat/flatpak-ide-support branch from a442ad7 to 59cb805 Compare January 19, 2025 16:46
@acolombier acolombier force-pushed the feat/flatpak-ide-support branch from 59cb805 to cd4654f Compare January 19, 2025 16:48
Copy link
Member

@pascalbreuninger pascalbreuninger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM overall, couple of small comments. Thanks for the contribution again!

Just as a reminder, we'll need to create a follow up PR to display VSCodium Insiders in DevPod Desktop.

Edit: linting failed

Name: config.IDECodiumInsiders,
DisplayName: "Codium Insiders",
Options: vscode.Options,
Icon: "https://devpod.sh/assets/codium_insiders.svg", // TODO to be uploaded
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is available now, you can remove the TODO :)

@@ -24,5 +24,6 @@ const (
IDEPositron IDE = "positron"
IDEMarimo IDE = "marimo"
IDECodium IDE = "codium"
IDECodiumInsiders IDE = "codium-insiders"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We'll have to expose this in DevPod Desktop as well, will create a follow up PR

@acolombier acolombier force-pushed the feat/flatpak-ide-support branch from 16bd37d to bf7710f Compare February 27, 2025 10:35
@acolombier
Copy link
Contributor Author

acolombier commented Feb 27, 2025

Apologies for the delay here @pascalbreuninger and thanks for your review. I have been using this version for the last month and found two more limitation addressed in 0bd8bc4

  • flatpak run doesn't fork correctly is the standard outputs are buffered (seems to be a bug upstream, haven't had a chace to dive too deep) - changes in pkg/ide/vscode/open.go
  • Proper support for customisation in VSCodium insiders - changes in pkg/ide/vscode/vscode.go

Let me know if it needs further attention, I'll squash the fixup otherwise! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants