Skip to content

Commit a0dff8a

Browse files
JacksonBurnsssun30
authored andcommitted
allow cat of regression diff to fail, print a warning instead
this will allow the regression test results to all be reported even if there is such a huge difference between the dynamic and baseline that the system utilities cannot print it see: #2316 (comment)
1 parent 1e4bdb9 commit a0dff8a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/CI.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -274,7 +274,7 @@ jobs:
274274
export FAILED=Yes
275275
fi
276276
echo "" # blank line so next block is interpreted as markdown
277-
cat "$regr_test-core.log"
277+
cat "$regr_test-core.log" || (echo "Dumping the whole log failed, please download it from GitHub actions. Here are the first 100 lines:" && head -n100 "$regr_test-core.log")
278278
echo "</details>"
279279
echo "<details>"
280280
if python-jl scripts/checkModels.py \
@@ -291,7 +291,7 @@ jobs:
291291
export FAILED=Yes
292292
fi
293293
echo "" # blank line so next block is interpreted as markdown
294-
cat "$regr_test-edge.log"
294+
cat "$regr_test-edge.log" || (echo "Dumping the whole log failed, please download it from GitHub actions. Here are the first 100 lines:" && head -n100 "$regr_test-core.log")
295295
echo "</details>"
296296
297297
# Check for Regression between Reference and Dynamic (skip superminimal)

0 commit comments

Comments
 (0)