Skip to content

Pin yq version#65

Merged
creydr merged 2 commits intofunctions-dev:mainfrom
creydr:install-yq-in-makefile
Apr 17, 2026
Merged

Pin yq version#65
creydr merged 2 commits intofunctions-dev:mainfrom
creydr:install-yq-in-makefile

Conversation

@creydr
Copy link
Copy Markdown
Collaborator

@creydr creydr commented Apr 17, 2026

The release workflow was failing because the Makefile uses yq to add version and commit labels to the installation manifest, but yq 4 was installed in the GitHub Actions environment (while we used yq 3 style in our Makefile).

This change adds a yq target to the Makefile that automatically downloads yq v3.4.1 from GitHub releases to the local bin/ directory, following the same pattern used for other tools like kustomize and operator-sdk.

The build-installer target now depends on yq and uses $(YQ) instead of the bare yq command to ensure the correct version is used.

creydr added 2 commits April 17, 2026 13:15
The release workflow was failing because the Makefile uses yq to add
version and commit labels to the installation manifest, but yq was not
installed in the GitHub Actions environment.

This change adds a yq target to the Makefile that automatically downloads
yq v3.4.1 from GitHub releases to the local bin/ directory, following the
same pattern used for other tools like kustomize and operator-sdk.

The build-installer target now depends on yq and uses $(YQ) instead of
the bare yq command to ensure the correct version is used.
@creydr creydr enabled auto-merge April 17, 2026 11:16
@creydr creydr disabled auto-merge April 17, 2026 11:44
@creydr creydr enabled auto-merge April 17, 2026 11:44
@creydr creydr added this pull request to the merge queue Apr 17, 2026
@creydr creydr changed the title Fix release workflow by downloading yq v3.4.1 to bin/ Pin yq version Apr 17, 2026
Merged via the queue into functions-dev:main with commit b59b14c Apr 17, 2026
11 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