Skip to content

Add alexpasmantier/television (tv) devcontainer feature#135

Merged
sebst merged 3 commits intomainfrom
copilot/add-alexpasmantier-television-feature
Mar 21, 2026
Merged

Add alexpasmantier/television (tv) devcontainer feature#135
sebst merged 3 commits intomainfrom
copilot/add-alexpasmantier-television-feature

Conversation

Copy link
Contributor

Copilot AI commented Mar 20, 2026

Adds a new community devcontainer feature to install tv, a fast terminal fuzzy finder written in Rust.

Changes

  • src/alexpasmantier-television/devcontainer-feature.json — Feature metadata; exposes a version option (default: latest)
  • src/alexpasmantier-television/install.sh — Downloads the tv binary from GitHub releases (tv-{VERSION}-{ARCH}-unknown-linux-gnu.tar.gz); resolves latest via GitHub API; supports amd64/arm64
  • test/alexpasmantier-television/test.sh — Verifies tv --version outputs television
  • .github/workflows/test.yaml — Adds alexpasmantier-television to both test-autogenerated and test-scenarios matrices

Usage

"features": {
    "ghcr.io/devcontainer-community/devcontainer-features/alexpasmantier-television:1": {}
}
Original prompt

This section details on the original issue you should resolve

<issue_title>Add alexpasmantier/television feature</issue_title>
<issue_description>Files:

src/alexpasmantier-television/devcontainer-feature.json
src/alexpasmantier-television/install.sh (make sure to set executable bit)
test/alexpasmantier-television/test.sh
.github/workflows/test.yaml

Releases:

https://github.com/alexpasmantier/television/releases</issue_description>

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


📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

Copilot AI changed the title [WIP] Add alexpasmantier/television feature Add alexpasmantier/television (tv) devcontainer feature Mar 20, 2026
Copilot AI requested a review from sebst March 20, 2026 17:07
@sebst sebst marked this pull request as ready for review March 21, 2026 16:33
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 fail

Copy link
Contributor Author

Copilot AI commented Mar 21, 2026

tests fail

Fixed in 732d7a3. There were two bugs:

  1. tar: tv: Not found in archive — the release tarball puts the binary inside a directory (tv-{VERSION}-{ARCH}-unknown-linux-gnu/tv), not at the root. Added a binaryPathInArchiveTemplate that expands to the correct full path, so --strip-components=1 strips the directory prefix.

  2. local: version: readonly variablegithub_get_tag_for_version declared local version="$2" but version is already readonly in install() before that call. Renamed the local parameter to _version.

Copilot AI requested a review from sebst March 21, 2026 17:09
@sebst sebst merged commit fbddcd7 into main Mar 21, 2026
176 of 185 checks passed
@sebst sebst deleted the copilot/add-alexpasmantier-television-feature branch March 21, 2026 17:19
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 alexpasmantier-television feature

2 participants