Skip to content

Commit 1223cea

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 6fb92d4 commit 1223cea

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
@@ -272,7 +272,7 @@ jobs:
272272
export FAILED=Yes
273273
fi
274274
echo "" # blank line so next block is interpreted as markdown
275-
cat "$regr_test-core.log"
275+
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")
276276
echo "</details>"
277277
echo "<details>"
278278
if python-jl scripts/checkModels.py \
@@ -289,7 +289,7 @@ jobs:
289289
export FAILED=Yes
290290
fi
291291
echo "" # blank line so next block is interpreted as markdown
292-
cat "$regr_test-edge.log"
292+
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")
293293
echo "</details>"
294294
295295
# Check for Regression between Reference and Dynamic (skip superminimal)

0 commit comments

Comments
 (0)