Skip to content

Commit c8cc778

Browse files
use official recommendation for parallel job output handling
1 parent fc9827e commit c8cc778

File tree

2 files changed

+31
-3
lines changed

2 files changed

+31
-3
lines changed

.github/workflows/verify.sh

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,13 +26,15 @@ cat module_list
2626
(echo "<project><modelVersion>4.0.0</modelVersion><groupId>temp</groupId><artifactId>temp</artifactId><version>1.0</version><packaging>pom</packaging><modules>"; cat module_list; echo "</modules></project>") > pom.xml
2727
cat pom.xml
2828

29-
# test compilation
29+
# use generated pom to test compilation
3030
mvn clean compile -T 1.5C -Dmaven.testSkip=true -Denforcer.skip -fae --fail-at-end 2>&1 | tee out
3131
cat out | grep 'rev20' | grep 's]' | { grep 'FAILURE' || true; } > errors
3232
if [[ $(cat errors | wc -l) -gt 0 ]]; then
3333
echo "Compilation errors found in the following libraries:"
3434
cat errors
3535
# send to GH output
36-
echo "failed_libraries=$(cat errors | tr '\n' ',')" > "${GITHUB_OUTPUT}"
36+
if [[ -f "${GITHUB_OUTPUT}" ]]; then
37+
echo "failed_libraries_${starting_letter}=$(cat errors | tr '\n' ',')" > "${GITHUB_OUTPUT}"
38+
fi
3739
fi
3840
echo "No compilation errors found"

.github/workflows/verify.yaml

Lines changed: 27 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,33 @@ jobs:
1818
letter: ['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z']
1919
fail-fast: false
2020
outputs:
21-
failed_libraries: ${{ steps.compile.outputs.failed_libraries }}
21+
# follows official example from https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/passing-information-between-jobs#using-job-outputs-in-a-matrix-job
22+
failed_libraries_a: ${{ steps.compile.outputs.failed_libraries_a }}
23+
failed_libraries_b: ${{ steps.compile.outputs.failed_libraries_b }}
24+
failed_libraries_c: ${{ steps.compile.outputs.failed_libraries_c }}
25+
failed_libraries_d: ${{ steps.compile.outputs.failed_libraries_d }}
26+
failed_libraries_e: ${{ steps.compile.outputs.failed_libraries_e }}
27+
failed_libraries_f: ${{ steps.compile.outputs.failed_libraries_f }}
28+
failed_libraries_g: ${{ steps.compile.outputs.failed_libraries_g }}
29+
failed_libraries_h: ${{ steps.compile.outputs.failed_libraries_h }}
30+
failed_libraries_i: ${{ steps.compile.outputs.failed_libraries_i }}
31+
failed_libraries_j: ${{ steps.compile.outputs.failed_libraries_j }}
32+
failed_libraries_k: ${{ steps.compile.outputs.failed_libraries_k }}
33+
failed_libraries_l: ${{ steps.compile.outputs.failed_libraries_l }}
34+
failed_libraries_m: ${{ steps.compile.outputs.failed_libraries_m }}
35+
failed_libraries_n: ${{ steps.compile.outputs.failed_libraries_n }}
36+
failed_libraries_o: ${{ steps.compile.outputs.failed_libraries_o }}
37+
failed_libraries_p: ${{ steps.compile.outputs.failed_libraries_p }}
38+
failed_libraries_q: ${{ steps.compile.outputs.failed_libraries_q }}
39+
failed_libraries_r: ${{ steps.compile.outputs.failed_libraries_r }}
40+
failed_libraries_s: ${{ steps.compile.outputs.failed_libraries_s }}
41+
failed_libraries_t: ${{ steps.compile.outputs.failed_libraries_t }}
42+
failed_libraries_u: ${{ steps.compile.outputs.failed_libraries_u }}
43+
failed_libraries_v: ${{ steps.compile.outputs.failed_libraries_v }}
44+
failed_libraries_w: ${{ steps.compile.outputs.failed_libraries_w }}
45+
failed_libraries_x: ${{ steps.compile.outputs.failed_libraries_x }}
46+
failed_libraries_y: ${{ steps.compile.outputs.failed_libraries_y }}
47+
failed_libraries_z: ${{ steps.compile.outputs.failed_libraries_z }}
2248
steps:
2349
- uses: actions/setup-java@v3
2450
with:

0 commit comments

Comments
 (0)