Skip to content

feat: add GitHub CLI (gh)#269

Merged
tungbq merged 1 commit into
mainfrom
feature/add-github-cli
Jul 13, 2026
Merged

feat: add GitHub CLI (gh)#269
tungbq merged 1 commit into
mainfrom
feature/add-github-cli

Conversation

@tungbq

@tungbq tungbq commented Jul 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds GitHub CLI (gh) to the toolkit, installed the same way as terraform/kubectl/helm — a pinned version downloaded directly from GitHub releases, no extra apt repo needed.
  • Wired into the existing version-check/version-sync automation (check_latest_version.py, update_latest_version.py, check_version_in_toolkit.sh) and the README tool table.
  • gh --version doesn't require auth, so it's added as a real smoke test in samples/run_sample.sh (same treatment as kubectl version/helm search), rather than left as documentation-only like AwsCLI/AzureCLI, which need live credentials to demonstrate anything.
  • New docs/usage/githubcli_usage.md follows the existing per-tool usage doc pattern (prerequisite, docs links, sample gh auth login + gh pr/gh issue usage).

Test plan

  • Docker Image CI builds cleanly and gh --version passes as part of the sample smoke test
  • Trivy gate passes
  • Spot-check gh auth login / gh repo list inside the built image

Installed the same way as terraform/kubectl/helm: a pinned version
downloaded directly from GitHub releases, no extra apt repo needed.

- toolkit_info.json / README / docs wired into the existing
  version-check and version-sync automation (check_latest_version.py,
  update_latest_version.py, check_version_in_toolkit.sh)
- `gh --version` needs no auth, so it's added as a real smoke test in
  samples/run_sample.sh (same treatment as kubectl/helm), unlike
  awscli/azurecli which stay documentation-only since they need live
  credentials to do anything meaningful
- New docs/usage/githubcli_usage.md follows the existing per-tool
  usage doc pattern
@tungbq tungbq merged commit aa194eb into main Jul 13, 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.

1 participant