Repository navigation
chore(config): add devcontainer setup - #154
etiennej70 wants to merge 6 commits into
Conversation
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Shared Docker state, stale PID ownership, and incomplete CA import can cause corruption, unrelated process termination, or failed proxy connectivity.
Get a fresh assessment by requesting another Copilot review.
Review effort: Balanced
Findings: 2
Open (6)
Use checkout-specific Docker volume names · New Validate PID ownership before managing stored process groups · New Normalize PEM certificates to CRT before updating trust store · New Capitalize the VS Code product name consistently · New Remove misleading supervisord troubleshooting guidance · New Reference application.yml instead of nonexistent app.yml · New
What changed in this PR
Adds a reproducible VS Code development container with automated services, CI validation, dependency updates, and contributor documentation.
Changes:
- Adds the Java 25 devcontainer and lifecycle scripts.
- Automates PostgreSQL, API, and documentation startup.
- Adds CI verification and weekly dependency updates.
| File | Description |
|---|---|
.devcontainer/Dockerfile |
Builds the development image and imports certificates. |
.devcontainer/certs/.gitkeep |
Preserves the certificate directory. |
.devcontainer/devcontainer-lock.json |
Locks devcontainer features. |
.devcontainer/devcontainer.json |
Configures tools, volumes, ports, and lifecycle commands. |
.devcontainer/post-create.sh |
Installs dependencies and prepares Maven. |
.devcontainer/post-start.sh |
Starts PostgreSQL, docs, and API services. |
.devcontainer/scripts/dev-services.sh |
Manages application and documentation processes. |
.devcontainer/scripts/export-host-ca-certs.sh |
Exports host CA certificates. |
.gitattributes |
Enforces platform-appropriate line endings. |
.github/dependabot.yml |
Adds weekly container dependency updates. |
.github/workflows/devcontainer.yml |
Builds and verifies the devcontainer in CI. |
.gitignore |
Excludes exported host certificates. |
docker-compose.yml |
Pins PostgreSQL and adds a health check. |
docs/src/contributing/development-setup.md |
Documents devcontainer usage and troubleshooting. |
src/main/resources/application-local.yml |
Disables Spring browser LiveReload locally. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
a35cb14 to
42f55c7
Compare
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Etienne JACQUOT <2057151+etiennej70@users.noreply.github.com>
…lize CA certs Co-authored-by: etiennej70 <2057151+etiennej70@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
cfa1fa2 to
828f8f1
Compare
|






What this PR Provides
Fixes
N/A
Review
The reviewer must double-check these points:
How to test
Initial state: You have Docker Desktop (Windows/macOS) or Docker Engine (Linux) installed, and VS Code with the "Dev Containers" extension.
Steps:
bash .devcontainer/scripts/dev-services.sh status all— you should see the app and docs services reported as runningExpected result: The container opens without errors, the API and docs are reachable at the URLs above, and code changes trigger an automatic reload.
Breaking changes
N/A