Skip to content

Commit b4b04a6

Browse files
ci: Fix check-changes job skipping over YAML changes. (#8377)
This commit fixes a copy/paste error from #8356 that resulted in the YAML detection logic of the `check-changes` job setting the wrong result for the job step's yaml changes output. This bug caused downstream jobs to not see that YAML files were altered at all in a PR, and YAML-specific jobs like the linter and zizmor passes would not be run. Signed-off-by: Philip Conrad <philip@chariot-chaser.net>
1 parent 3c5b7c6 commit b4b04a6

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/workflows/pull-request.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ jobs:
7171
wasm_result=$(jq -r '.changes.wasm // false' opa_result.json)
7272
docs_result=$(jq -r '.changes.docs // false' opa_result.json)
7373
rego_result=$(jq -r '.changes.rego // false' opa_result.json)
74-
yaml_result=$(jq -r '.changes.rego // false' opa_result.json)
74+
yaml_result=$(jq -r '.changes.yaml // false' opa_result.json)
7575
7676
echo "go=${go_result}" >> $GITHUB_OUTPUT
7777
echo "wasm=${wasm_result}" >> $GITHUB_OUTPUT

0 commit comments

Comments
 (0)