Skip to content

Commit da04fd1

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 7aceea2 commit da04fd1

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

0 commit comments

Comments
 (0)