New devcontainers onboarding#1189
Conversation
This shouldn't break anyones workflow too bad since they can either use `podman pod start dsekweb-services to keep existing setp or switch to podman-compose for new installs that want to use podman
Tested with podman-compose
eedcb7e to
aed5e28
Compare
| @@ -0,0 +1,36 @@ | |||
| services: | |||
There was a problem hiding this comment.
isn't this missing our meilisearch and poppler services? I feel like it could be useful to have those even in a dev environment.
There was a problem hiding this comment.
I intentionally left them out, but we can discuss if that's the right choice. In my opinion, most common development tasks don't need them. I'm choosing to prioritize keeping the default setup small over having a production-like development environment. Our current setup is also a compromise. It doesn't include scheduler-service or shlink in the default dev workflow and minio requires extra configuration.
There was a problem hiding this comment.
Reasonable, but I do believe the search function is highly useful when debugging mostly anything, since it allows a user to easily traverse the webpage, and make simple queries as to what data is loaded (Members, Positions and so on). With that said Meilisearch requires to be seeded when started though which I don't believe most people do, so it may be unnecessary as you say.
There was a problem hiding this comment.
I added a second configuration that includes search, but without is default.
d715c9d to
cf91779
Compare
Tomas-Ka
left a comment
There was a problem hiding this comment.
Haven't tested the devcontainer integration, but the image itself works great
|
Works om my machine! VSC on plain Arclonüx |
Good for compatibility with default podman configuration on fedora
f1455e7 to
f064c8d
Compare
🧩 Summary
Create a new way to set up a development environment based only on containers and with support for devcontainers.
🔗 Related issues (if any)
Closes #570
Closes #566
💬 Other information
There is more to do in terms of documentation. We need to rewrite Getting started web if this is merged.