Skip to content

fix(ci-action): install git in Dockerfile so lingo.dev ci works out…#2096

Merged
moygospadin merged 2 commits into
lingodotdev:mainfrom
louisguitton:fix/dockerfile-install-git
May 27, 2026
Merged

fix(ci-action): install git in Dockerfile so lingo.dev ci works out…#2096
moygospadin merged 2 commits into
lingodotdev:mainfrom
louisguitton:fix/dockerfile-install-git

Conversation

@louisguitton
Copy link
Copy Markdown
Contributor

@louisguitton louisguitton commented May 27, 2026

Summary

Install git in the lingodotdev/ci-action Docker image so lingo.dev ci works in runners that do not already provide it (e.g. GitLab CI using the image directly).

Changes

  • Dockerfile: add RUN apk add --no-cache git after the node:20.12.2-alpine base image.
  • .changeset/fix-dockerfile-install-git.md: patch-level changeset for the lingo.dev package.

Testing

Business logic tests added:

  • N/A — Dockerfile-only change. No existing image-level test harness in the repo, and there is no CLI code path to unit-test.
  • Verified locally by building the image and running lingo.dev ci:
    docker build -t ci-action-local .
    docker run --rm -v "$PWD:/work" -w /work ci-action-local \
      sh -c 'git --version && npx lingo.dev@latest ci --help'
    Before the change: /bin/sh: git: not found, exit 127.
    After the change: git version 2.x and the CLI help prints.
    
    

Visuals

  • N/A — no UI surface.

Checklist

  • Changeset added (if version bump needed)
  • Tests cover business logic (not just happy path)
  • No breaking changes (or documented below)

Closes #2074

Summary by CodeRabbit

  • Bug Fixes
    • Updated the Docker image to install git as a system dependency. The CI action now runs successfully on CI runners that don't have git pre-installed, ensuring compatibility across different environments.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 27, 2026

📝 Walkthrough

Walkthrough

The pull request adds git as a system dependency to the lingodotdev/ci-action Docker image and documents the fix in a changeset. The Dockerfile now installs git via apk add --no-cache git, enabling the lingo.dev ci command to run on CI runners (e.g., GitLab CI) that do not include git by default.

Changes

Docker Image Git Dependency Fix

Layer / File(s) Summary
Docker image git installation and release documentation
Dockerfile, .changeset/fix-dockerfile-install-git.md
Dockerfile installs git via apk in Alpine Node image; changeset documents patch release for lingodotdev/ci-action to enable ci command on non-GitHub runners.

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A rabbit hops through Docker files,
Finds git was hiding all this while!
One line of apk add to save the day—
Now CI runs smoothly, hip-hop-hooray! 🐰📦

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main fix: adding git to the Dockerfile for the CI action image, enabling lingo.dev ci to work in runners without git.
Linked Issues check ✅ Passed The pull request directly implements the proposed fix from issue #2074: adding git to the Dockerfile via apk add, enabling lingo.dev ci to work outside GitHub runners.
Out of Scope Changes check ✅ Passed All changes are directly scoped to issue #2074: the Dockerfile modification to add git and the changeset file documenting the patch update.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The PR description follows the template structure with all required sections completed: Summary, Changes, Testing, Visuals, and Checklist are all present and properly filled out.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@moygospadin moygospadin merged commit 7d803d2 into lingodotdev:main May 27, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docker image lingodotdev/ci-action missing git, breaks npx lingo.dev ci outside GitHub runners

2 participants