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

Closed
wants to merge 1 commit into 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!

@dsyer
Copy link
Member

dsyer commented Jul 29, 2024

Ping @jdubois again. Any input most welcome.

@jdubois
Copy link
Contributor

jdubois commented Jul 30, 2024

I just tried it in Codespaces and it didn't work. I didn't have a popup asking me which Devcontainer to use, and as a result it started with the default Java configuration (using JDK 11).

@jdubois
Copy link
Contributor

jdubois commented Jul 30, 2024

Then this configuration is quite old, let me have a look at modernising it, and probably that would also help for this issue.

@dsyer
Copy link
Member

dsyer commented Aug 21, 2024

Can you rebase and force push to include the changes from #1623?

@dsyer
Copy link
Member

dsyer commented Sep 12, 2024

Timeout on rebase. If someone wants to do it just send another PR.

@dsyer dsyer closed this Sep 12, 2024
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.

4 participants