Repository navigation
feat(container): add Docker and Podman support - #1037
gowrishacv wants to merge 5 commits into
Conversation
code-review-graph reviewOverall risk: 0.00 (LOW) — 0 changed function(s)/class(es), 0 affected flow(s), 0 test gap(s) Token savings: this graph-backed report used ~66,028 fewer tokens (~100%) than reading every changed file in full (estimated, chars/4 approximation). Powered by code-review-graph — local-first analysis; no code leaves the CI runner. |
|
Thank you for this PR. I tested Before I can merge it, please fix these:
After you push the fix, I will test it again. |
|
@tirth8205 Both requested changes are pushed in 7d3af0d.
Validation on an Apple Silicon Mac:
Could you please retest the updated image with your Docker Desktop workflow, especially |
There was a problem hiding this comment.
🟡 Changes recommended
The documented bind mount can deny repository access on SELinux-enforcing hosts, so all four README translations need guidance on the :z mount option.
1 open issue
What changed in this PR
Adds Docker and Podman support for running the code-review-graph CLI from a locally built image, addressing #232.
Changes:
- Adds a multi-stage Python 3.12 image that installs repository source, includes Git, and defaults to a non-root user.
- Documents container builds, writable repository mounts, and user mapping in English and four translations.
- Explains graph persistence and container-path constraints when switching to host commands.
| File | Description |
|---|---|
| README.zh-CN.md | Adds Simplified Chinese container usage instructions. |
| README.md | Documents Docker and Podman commands and graph persistence. |
| README.ko-KR.md | Adds Korean container usage instructions. |
| README.ja-JP.md | Adds Japanese container usage instructions. |
| README.hi-IN.md | Adds Hindi container usage instructions. |
| Dockerfile | Builds and installs the CLI with Git and a non-root runtime. |
Review effort: Balanced
| code-review-graph:local build | ||
| ``` | ||
|
|
||
| The graph persists in `.code-review-graph/` in the mounted repository and stores |
The review on tirth8205#1037 asked for the `:z` mount option on SELinux hosts in the README and its four translations. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

What changed
Adds a Dockerfile that builds and installs the current repository source using
python:3.12-slim. The final image includes Git and defaults to the non-rootcrguser (UID/GID 10001).Adds Docker and Podman instructions to the English README and all four translations, including user mapping for writable repository mounts and persisted graph data.
No graph is generated during image construction. GHCR publishing remains outside this PR.
Related issue
Closes #232
Validation
uid=10001(crg) gid=10001(crg).git version 2.47.3.Both Docker and Podman produced:
A separate container read the persisted graph with
status:Repository checks
git diff --check: passed.The test warning concerns Starlette’s deprecated use of
httpxin its test client.