Skip to content

Add lazygit devcontainer feature - #108

Merged
sebst merged 4 commits into
mainfrom
copilot/add-lazygit-feature
Mar 19, 2026
Merged

sebst merged 4 commits into
mainfrom
copilot/add-lazygit-feature

Conversation

Copilot AI commented Mar 19, 2026 •

Copy link
Copy Markdown
Contributor

Adds a lazygit devcontainer feature that installs the lazygit terminal UI for git from GitHub releases, following the established pattern in this repo.

Changes

  • src/lazygit/devcontainer-feature.json — Feature metadata; exposes a version option (default: latest)
  • src/lazygit/install.sh — Downloads and installs the lazygit binary from jesseduffield/lazygit GitHub releases; maps Debian arch (amd64→x86_64, arm64→arm64, armhf→armv6, i386→x86) to match lazygit's release naming convention (lazygit_{version}_Linux_{arch}.tar.gz)
  • test/lazygit/test.sh — Validates lazygit --version runs successfully
  • .github/workflows/test.yaml — Adds lazygit to both test-autogenerated and test-scenarios CI matrices
Original prompt

This section details on the original issue you should resolve

<issue_title>Add lazygit feature</issue_title>
<issue_description>build on implementation in #104 </issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: sebst <592313+sebst@users.noreply.github.com>
Copilot AI changed the title [WIP] Add lazygit feature Add lazygit devcontainer feature Mar 19, 2026
Copilot AI requested a review from sebst March 19, 2026 09:29
@sebst
sebst marked this pull request as ready for review March 19, 2026 09:32

@sebst sebst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tests failing

Co-authored-by: sebst <592313+sebst@users.noreply.github.com>

Copilot AI commented Mar 19, 2026

Copy link
Copy Markdown
Contributor Author

tests failing

Fixed in f41478a. Two issues were causing the test failures:

  1. Test assertion (test/lazygit/test.sh): lazygit --version outputs commit=..., version=x.x.x, os=..., arch=... — it doesn't include the word "lazygit". Changed the grep from grep 'lazygit' to grep 'version='.

  2. readonly variable error (install.sh): github_get_tag_for_version used local version="$2", which conflicted with the readonly version set earlier in install(). Renamed the local to ver to eliminate the naming clash.

Copilot AI requested a review from sebst March 19, 2026 10:18
@sebst
sebst merged commit 5a07a45 into main Mar 19, 2026
156 of 157 checks passed
@sebst
sebst deleted the copilot/add-lazygit-feature branch March 19, 2026 11:11
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.

Add lazygit feature

2 participants