Skip to content

Add mikefarah/yq to ci-tools image#89

Merged
lex57ukr merged 1 commit intomainfrom
83-add-yq-to-ci-tools
Mar 14, 2026
Merged

Add mikefarah/yq to ci-tools image#89
lex57ukr merged 1 commit intomainfrom
83-add-yq-to-ci-tools

Conversation

@lex57ukr
Copy link
Copy Markdown
Contributor

Summary

BATS tests that source keystone runtime scripts using yq currently skip in ci-tools because the binary is missing. This adds mikefarah/yq to the image so shell-level unit test coverage is no longer incomplete.

Related Issues

Refs #83

Changes

  • Add yq binary install to Dockerfile with per-arch SHA-256 verification (same pattern as shfmt/actionlint/hadolint)
  • Add resolve_yq() to resolve.sh using GitHub native release digests
  • Add yq build args to compose.yaml and version check to verify.sh
  • Seed versions.lock with yq v4.52.4

Verification

make resolve TOOLS="yq", make build, make verify, and make lint all pass.

Install the standalone Go binary from GitHub releases with per-arch
SHA-256 verification, following the same pattern as shfmt/actionlint/hadolint.
This unblocks BATS tests that source keystone runtime scripts using yq.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@lex57ukr lex57ukr added the enhancement New feature or request label Mar 14, 2026
@lex57ukr lex57ukr enabled auto-merge (squash) March 14, 2026 17:46
@lex57ukr lex57ukr merged commit 32d8916 into main Mar 14, 2026
7 checks passed
@lex57ukr lex57ukr deleted the 83-add-yq-to-ci-tools branch March 14, 2026 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant