Don't wrap build matrix in "include{}" to avoid hard coding into the …

…script.
NVIDIA · May 30, 2023 · 606fdd8 · 606fdd8
1 parent 958fa09
commit 606fdd8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.github/workflows/test_new_workflow.yml b/.github/workflows/test_new_workflow.yml
@@ -31,7 +31,7 @@ jobs:
         id: compute-matrix
         run: |
           cat ./ci/matrix.yaml
-          BUILD_MATRIX=$(yq -o=json ./ci/matrix.yaml | jq -c '[.include[] | . as $o | {std: .std[]} + del($o.std)] | {include: . }' )
+          BUILD_MATRIX=$(yq -o=json ./ci/matrix.yaml | jq -c '[.include[] | . as $o | {std: .std[]} + del($o.std)]' )
           echo "BUILD_MATRIX=$BUILD_MATRIX" | tee -a "$GITHUB_OUTPUT"
           PER_CUDA_MATRIX=$(./ci/get_groups.sh $BUILD_MATRIX cuda)
           echo "PER_CUDA_MATRIX=$PER_CUDA_MATRIX" | tee -a "$GITHUB_OUTPUT"

diff --git a/ci/get_groups.sh b/ci/get_groups.sh
@@ -13,6 +13,6 @@ fi
 input_json="$1"
 field="$2"
 
-output=$(echo $input_json | jq -L . -c --arg field "$field" 'include "group_by_field"; .include | group_by_field($field)')
+output=$(echo $input_json | jq -L . -c --arg field "$field" 'include "group_by_field"; group_by_field($field)')
 echo $output