Skip to content

Commit 89d3f8f

Browse files
[CONFIG] Improve template action feedback slightly (- WIP #338 -)
Changes in file .github/actions/run-minimal-acceptance-tests/action.yml: * improved outputs a bit * related work Changes in file .github/workflows/CI-MATs.yml: * minor changes to format
1 parent 9cd25d8 commit 89d3f8f

File tree

2 files changed

+31
-8
lines changed

2 files changed

+31
-8
lines changed

.github/actions/run-minimal-acceptance-tests/action.yml

Lines changed: 28 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,30 @@ outputs:
3535
mats_status:
3636
description: "The outcome of evaluating the MATs steps"
3737
value: ${{ steps.mats_outcome.outcome || 'cancelled' }}
38+
mats_build_status:
39+
description: "The outcome of the MATs Build Group"
40+
value: ${{ steps.mats_build.outcome || 'cancelled' }}
41+
mats_bootstrap_status:
42+
description: "The outcome of the MATs Bootstrap Group"
43+
value: ${{ steps.mats_bootstrap.outcome || 'cancelled' }}
44+
mats_basic_status:
45+
description: "The outcome of the MATs Basic Group"
46+
value: ${{ steps.mats_basic.outcome || 'cancelled' }}
47+
mats_say_status:
48+
description: "The outcome of the MATs SAY Group"
49+
value: ${{ steps.mats_say.outcome || 'cancelled' }}
50+
mats_hear_status:
51+
description: "The outcome of the MATs HEAR Group"
52+
value: ${{ steps.mats_hear.outcome || 'cancelled' }}
53+
mats_usage_status:
54+
description: "The outcome of the MATs Usage Group"
55+
value: ${{ steps.mats_usage.outcome || 'cancelled' }}
56+
mats_coverage_status:
57+
description: "The outcome of the MATs Coverage Group"
58+
value: ${{ steps.mats_coverage.outcome || 'cancelled' }}
59+
mats_doctests_status:
60+
description: "The outcome of the MATs DocTests Group"
61+
value: ${{ steps.mats_doctests.outcome || 'cancelled' }}
3862
artifact-name:
3963
description: "The uploaded artifact-name"
4064
value: ${{ steps.output_artifact_name.outputs.artifact-name }}
@@ -191,11 +215,11 @@ runs:
191215
shell: bash
192216
run: |
193217
if [[ "${{ steps.mats_outcome.outcome }}" == "success" ]] ; then
194-
printf "%s\n" " * :ballot_box_with_check: MATS succeeded with python version ${PYTHON_VERSION} for [${BUILD_SHA}](https://github.com/reactive-firewall/multicast/commit/${BUILD_SHA})" > "$MATS_STEP_SUMMARY"
195-
printf "%s\n" " :ballot_box_with_check: MATS succeeded with python version ${PYTHON_VERSION}" >> "$GITHUB_STEP_SUMMARY"
218+
printf "%s\n" " * :ballot_box_with_check: MATS succeeded with python version \`${PYTHON_VERSION}\` for [${BUILD_SHA}](https://github.com/reactive-firewall/multicast/commit/${BUILD_SHA})" > "$MATS_STEP_SUMMARY"
219+
printf "%s\n" " :ballot_box_with_check: MATS succeeded with python version \`${PYTHON_VERSION}\`" >> "$GITHUB_STEP_SUMMARY"
196220
else
197-
printf "%s\n" " * :x: ~MATS succeeded with python version ${PYTHON_VERSION} for ${BUILD_SHA}~" > "$MATS_STEP_SUMMARY"
198-
printf "%s\n" " :x: ~MATS succeeded with python version ${PYTHON_VERSION}~" >> "$GITHUB_STEP_SUMMARY"
221+
printf "%s\n" " * :x: ~MATS succeeded with python version \`${PYTHON_VERSION}\` for \`${BUILD_SHA}\`~" > "$MATS_STEP_SUMMARY"
222+
printf "%s\n" " :x: ~MATS succeeded with python version \`${PYTHON_VERSION}\`~" >> "$GITHUB_STEP_SUMMARY"
199223
fi
200224
if [[ "${{ steps.mats_build.outcome }}" == "success" ]] ; then
201225
printf "%s\n" " * :ballot_box_with_check: Minimal Build Testing succeeded" >> "$MATS_STEP_SUMMARY"

.github/workflows/CI-MATs.yml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -197,12 +197,11 @@ jobs:
197197
echo "## :building_construction: Build" >> "${MATS_COMMENT_BODY}"
198198
echo "" >> "${MATS_COMMENT_BODY}"
199199
if [[ "${{ needs.check_build.outputs.should_run }}" == "true" ]]; then
200-
BUILD_SHA="${{ needs.check_build.outputs.sha }}"
201200
echo " :ballot_box_with_check: BUILD \`${{ needs.check_build.outputs.trigger_id }}\` succeeded with commit [${BUILD_SHA}](https://github.com/reactive-firewall/multicast/commit/${BUILD_SHA})" >> "${MATS_COMMENT_BODY}"
202-
echo "build_success=true" >> "${MATS_COMMENT_BODY}"
201+
echo "build_success=true" >> "$GITHUB_OUTPUT"
203202
else
204203
echo " :x: BUILD ${{ needs.check_build.outputs.trigger_id }} failed" >> "${MATS_COMMENT_BODY}"
205-
echo "build_success=false" >> "${MATS_COMMENT_BODY}"
204+
echo "build_success=false" >> "$GITHUB_OUTPUT"
206205
fi
207206
echo "" >> "${MATS_COMMENT_BODY}"
208207
echo "## :1234: Minimal Acceptance Testing" >> "${MATS_COMMENT_BODY}"
@@ -223,7 +222,7 @@ jobs:
223222
echo "## :construction: Build" >> "${MATS_COMMENT_BODY}"
224223
echo "" >> "${MATS_COMMENT_BODY}"
225224
if [[ "${{ needs.check_build.outputs.should_run }}" == "true" ]]; then
226-
echo " :ballot_box_with_check: BUILD \`${{ needs.check_build.outputs.trigger_id }}\` succeeded with commit ${{ needs.check_build.outputs.sha }}" >> "${MATS_COMMENT_BODY}"
225+
echo " :ballot_box_with_check: BUILD \`${{ needs.check_build.outputs.trigger_id }}\` succeeded with commit [${BUILD_SHA}](https://github.com/reactive-firewall/multicast/commit/${BUILD_SHA})" >> "${MATS_COMMENT_BODY}"
227226
echo "build_success=true" >> "$GITHUB_OUTPUT"
228227
else
229228
echo " :x: BUILD ${{ needs.check_build.outputs.trigger_id }} failed" >> "${MATS_COMMENT_BODY}"

0 commit comments

Comments
 (0)