Create Devcontainer config#4793
Conversation
|
@TimWolla it's all ready for you now |
|
I don't know anything about containerization and don't have any strong opinions about this. |
|
Thank you. I've tried spawning a codespace from your repository, connected to it via VS Code and ran It appears the Also, I think it would be nice to avoid the additional layer of Docker within a Codespace / dev container. Would it perhaps work to directly install Java + PHP in the Codespace at creation time? |
|
I'm sure I can find a way to avoid the extra Docker layer. No idea on the other problem though - it's not happening on my end with a fresh Codespace. |
|
Do I see correctly that Why is a devcontainer with a web server needed at all, when the |
|
@cweiske The Devcontainer spec is not specific to VS Code, though it was probably implemented there first as it's a Microsoft invention. PhpStorm supports it as well, but more importantly, GitHub Codespaces can use it to spawn a pre-customized browser-based dev environment based on VS Code. Sorry I've let this languish; life priorities got in the way and it slipped my mind. |
Remove the hostRequirements configuration that specified 4 CPUs and 16GB of memory. This allows the devcontainer to use default resource allocations instead of requiring elevated system resources.
The symbolic link creation for doc-base's language source directory is no longer needed and has been removed from the post-create initialization script.
…te script" This reverts commit 0b1aa64.
|
Thank you for proposing this idea @garrettw, and I am sorry this went stale for so long. I support this, in fact Is support any initiative that helps newcomers to get started more easily. And I believe this solutions also everyone to more quickly peek at new changes that are packed in a PR. It took me a while (as seen above) to feel satisfied with the setup. The original setup was using docker-in-docker, which didn't feel right to me as all the tools should already be available in devcontainer. The new setup is using PHP built-in webserver directly, built on Trixie which is the latest version of Debian. And does have PHP 8.4 included. Launch scripts are available to quickly build without the need to really understand the build process. Similar to how After getting an initial version working, I noticed it was.. slow. Which is why I got side tracked to fix php/doc-base#300. Which not only improved the dev container, but also give a huge improvement to using regular builds. |
TimWolla
left a comment
There was a problem hiding this comment.
Did a superficial review, but didn't see any obvious issues. Given the number of commits, I trust that you tested this. Thank you!
|
@jordikroon Awesome! Thank you so much for picking up the torch and getting this over the finish line! |
As requested by @TimWolla.