-
-
Notifications
You must be signed in to change notification settings - Fork 966
Add Devcontainer support #2488
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
base: main
Are you sure you want to change the base?
Add Devcontainer support #2488
Conversation
@dunglas do you know if I can in the php container point the php to |
So in 3.1.x path was "/srv/app/", in 3.2.x it was changed to "/app/" and simplified. In the last commit I changed it to "/srv/api/". |
As I mentioned in the symfony-docker project, i not in favor of this addition. |
@maxhelias can you elaborate on the "recipe"? I'm not familiar with this terminology |
@ili101 you can see more information here : https://symfony.com/doc/current/quick_tour/flex_recipes.html#flex-recipes-and-aliases |
598b25e
to
bf4a8c3
Compare
@maxhelias in response to your feedback here and on dunglas/symfony-docker#523
I have removed my opinions/examples in the
I think this is more appropriate for a template than a receipt for two reasons: Another benefit as we are on GitHub is Codespaces. Basically, if you have Devcontainer setup on the repo itself you can open it in the browser. So for example you want to check a bug or present API-Platform somewhere, all you need is a browser and you have a running environment, IDE, and a debuger. If you want to check it out:
|
Updated and squished all commits. |
My 2cents on this feature request. As I can maybe understand to rely on symfony/flex for this kind of things for dunglas/symfony-docker repository ; At least, those lines which concerned /srv/app -> /srv/api to avoid wasted time/energy. Yes, I'm not okay to enable xdebug by default ; but I'll be really happy if it can be commented out and not totally erased. I mean, you embrace tons-of standards and how those things going in industry. By this way, you are some light-in-the-dark for developers without active mentoring or well formed team and who keep eyes open on your stuff 💓 Please, DevContainer extension is this kind of welcome-things for quickly learn / play / discover / (work?). @ili101: Thanks to publish your retex and avoid this 2days-trap 👍 Doux-jésus, je ne suis pas client de la hype ou des annonces marketing, m'enfin être DevContainer compliant, c'est du slogan tout fait non ? Sans vouloir jouer sur l'éloquance, c'est un coup à finir couplé avec sa persistance ces broutilles 😅 ./me little disapointed about this 'not-already' merged request, @ili101 will maybe not rebase forever his/her branch :/ |
Removed Xdebug config for VSCode (there are no longer any VSCode specific configs/files left). To setup Xdebug on VSCode
environment:
# See https://xdebug.org/docs/all_settings#mode
XDEBUG_MODE: debug
# Enable the step debugger
XDEBUG_SESSION: 1
# Override api/docker/php/conf.d/app.dev.ini
XDEBUG_CONFIG: "client_host=localhost"
|
My bad, as I said, I was far from keyboard ^^' Devcontainer is closed source and seems Microsoft thing, I will not using this. (I'm using codium) Anyway, it didn't change my point of view for this thing : Cheers |
Actually, it's an open-source standard created by Microsoft and supported by VSCode and PhpStorm and some other tools. I assume you are referring to https://vscodium.com/. I see the VSCodium community is upset that while VSCode and the Devcontainer specification are open source the VSCode remote server implementation is not VSCodium/vscodium#1229 VSCodium/vscodium#1696. So the VSCodium team needs to make their own implementation or push Microsoft to open VSCode remote microsoft/vscode-remote-release#1925 (top 10 most 👍 issue, so maybe at some point?) |
#2472