Skip to content

Commit f65c6ab

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 8824dad commit f65c6ab

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
@@ -296,7 +296,7 @@ jobs:
296296
export FAILED=Yes
297297
fi
298298
echo "" # blank line so next block is interpreted as markdown
299-
cat "$regr_test-core.log"
299+
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")
300300
echo "</details>"
301301
echo "<details>"
302302
if python-jl scripts/checkModels.py \
@@ -313,7 +313,7 @@ jobs:
313313
export FAILED=Yes
314314
fi
315315
echo "" # blank line so next block is interpreted as markdown
316-
cat "$regr_test-edge.log"
316+
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")
317317
echo "</details>"
318318
319319
# Check for Regression between Reference and Dynamic (skip superminimal)

0 commit comments

Comments
 (0)