Skip to content

Commit f482cd0

Browse files
authored
Merge pull request #10466 from haskell/mergify/bp/3.12/pr-10462
kill a `continue-on-error` (backport #10462)
2 parents 61d6947 + ef83f18 commit f482cd0

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

.github/workflows/validate.yml

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -239,6 +239,12 @@ jobs:
239239
echo End "$test"
240240
done
241241
exit $rc
242+
# The above ensures all the tests get run, for a single platform+ghc.
243+
# Trying to ensure they run for *all* combinations but still fail
244+
# at the end seems to be extremely difficult at best. It's doable,
245+
# but it requires a continuously growing stack of conditions and
246+
# one possibly nightmarish final conditional. 'fail-fast' gets us
247+
# partway there, at least, but is still imperfect.
242248

243249
validate-old-ghcs:
244250
name: Validate old ghcs ${{ matrix.extra-ghc }}
@@ -298,8 +304,9 @@ jobs:
298304
- name: "Validate lib-suite-extras --extra-hc ghc-${{ matrix.extra-ghc }}"
299305
env:
300306
EXTRA_GHC: ghc-${{ matrix.extra-ghc }}
301-
continue-on-error: true
302307
run: sh validate.sh ${{ env.COMMON_FLAGS }} --lib-only -s lib-suite-extras --extra-hc "${{ env.EXTRA_GHC }}"
308+
# See the comment above about running all tests but still failing if one
309+
# of them does; it also applies here.
303310

304311
build-alpine:
305312
name: Build statically linked using alpine

0 commit comments

Comments
 (0)