Skip to content

Commit fadb72d

Browse files
Claudeclaude
andcommitted
CI - IMPROVEMENT - Log the Julia change-detection verdict to the job log
The detector wrote its decision only to the run summary, so the job log showed the script and nothing else. That makes "why did the suite run?" unanswerable from the place people look first. Echo the verdict and the matched files to stdout as well, and list the files considered when the suite is skipped. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019tFBCa5iBXyb7xp8LxqBJx
1 parent 673a3a0 commit fadb72d

1 file changed

Lines changed: 20 additions & 13 deletions

File tree

.github/workflows/test.yaml

Lines changed: 20 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,14 @@ jobs:
4141
run: |
4242
set -euo pipefail
4343
44+
# Report the decision to the job log as well as the run summary, so
45+
# `why did/didn't this run?` is answerable from either one.
46+
say() { echo "$1"; echo "$1" >> "$GITHUB_STEP_SUMMARY"; }
47+
4448
# Pushes to main/develop always run: every merged commit gets verified.
4549
if [ "${{ github.event_name }}" != "pull_request" ]; then
4650
echo "julia=true" >> "$GITHUB_OUTPUT"
47-
echo "Not a pull request — running the full test suite." >> "$GITHUB_STEP_SUMMARY"
51+
say "Not a pull request — running the full test suite."
4852
exit 0
4953
fi
5054
@@ -58,18 +62,21 @@ jobs:
5862
5963
relevant=$(printf '%s\n' "$files" | grep -Ev "$not_tested" | grep -E "$tested" || true)
6064
61-
{
62-
echo "### Julia change detection"
63-
echo
64-
if [ -n "$relevant" ]; then
65-
echo "Running the test suite. Matched files:"
66-
echo '```'
67-
echo "$relevant"
68-
echo '```'
69-
else
70-
echo "No Julia code, test data or example decks changed — skipping the test suite."
71-
fi
72-
} >> "$GITHUB_STEP_SUMMARY"
65+
say "### Julia change detection"
66+
say ""
67+
if [ -n "$relevant" ]; then
68+
say "Running the test suite. Matched files:"
69+
say '```'
70+
say "$relevant"
71+
say '```'
72+
else
73+
say "No Julia code, test data or example decks changed — skipping the test suite."
74+
say ""
75+
say "Files considered:"
76+
say '```'
77+
say "$files"
78+
say '```'
79+
fi
7380
7481
if [ -n "$relevant" ]; then
7582
echo "julia=true" >> "$GITHUB_OUTPUT"

0 commit comments

Comments
 (0)