Skip to content

Add lazygit devcontainer feature#108

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

Add lazygit devcontainer feature#108
sebst merged 4 commits intomainfrom
copilot/add-lazygit-feature

Conversation

Copy link
Contributor

Copilot AI commented Mar 19, 2026

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
Copy link
Member

@sebst sebst left a comment

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>
Copy link
Contributor Author

Copilot AI commented Mar 19, 2026

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