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

Support for JetBrains IDEs in Dev Container #1514

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

trukhinyuri
Copy link

Currently, support for Dev Containers in the project is exclusively oriented towards users of the vscode editor, which does not allow for development in an IDE.
Specifically, the devcontainer metadata contains proprietary commands that are only compatible with VSCode.
The devcontainer.json also uses bind, which is not compatible with using the project on remote Docker servers (in this case, docker volumes should be used, as one cannot assume that the host is always exclusively owned by the user).

To address this, subfolders for jetbrains and vscode were created (fully in accordance with the containers.dev specification). In the jetbrains folder, all specified scenarios have been corrected.
The vscode folder retains the original configuration.

When building a devcontainer, both IntelliJ IDEA 2024.1 and vscode can, in the case of multiple devcontainer.json files in the project, ask the user which metadata file to use, thus editor users will choose VSCode, while IDE users will opt for IntelliJ IDEA.
image

… run on a remote Docker server where the bind command does not allow the devcontainer to start).
@pivotal-cla
Copy link

@trukhinyuri Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-cla
Copy link

@trukhinyuri Thank you for signing the Contributor License Agreement!

@dsyer
Copy link
Member

dsyer commented May 16, 2024

It would be good to get someone from Microsoft to look at this (@jdubois or @roryp maybe?) to make sure that the vscode files still work, and that the experience isn't degraded in Codespaces (which was the main target originally). My impression from trying it is that Codespaces does not look in the subdirectory, so it gets a lot of things wrong.

(I also think you should be careful not to throw shade on vscode in your comments, but it's possible that's just a non-native English speaker making an honest mistake, so I might have drawn the wrong conclusion.)

@jdubois
Copy link
Contributor

jdubois commented May 16, 2024

Thanks for pinging me @dsyer ! Yes the main idea was to have it work with Codespaces, as everyone has 60 hours free per month, it helps a lot for small contributions to OSS projects, so this needs to continue working.
Anyway, the current configuration needs to be updated, I have much better devcontainer configurations now :-)
I'll have a look with my team at Microsoft!

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.

None yet

4 participants