Skip to content

Commit c5ca57c

Browse files
committed
feat: problem matchers for tests (#9552)
Makes it a bit easier to see which test failed and where
1 parent d3da7bb commit c5ca57c

File tree

4 files changed

+36
-24
lines changed

4 files changed

+36
-24
lines changed

.github/workflows/bors.yml

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -262,12 +262,15 @@ jobs:
262262
lake exe shake
263263
264264
- name: test mathlib
265-
id: test
266-
run: |
267-
# Tests use parts of ProofWidgets not imported by Mathlib.
268-
# Ensure everything has been built.
269-
lake build ProofWidgets
270-
make -j 8 test
265+
uses: liskin/gh-problem-matcher-wrap@v2
266+
with:
267+
linters: gcc
268+
run: |
269+
bash -o pipefail -c "
270+
# Tests use parts of ProofWidgets not imported by Mathlib.
271+
# Ensure everything has been built.
272+
lake build ProofWidgets
273+
make -j 8 test"
271274
272275
- name: lint mathlib
273276
id: lint

.github/workflows/build.yml

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -269,12 +269,15 @@ jobs:
269269
lake exe shake
270270
271271
- name: test mathlib
272-
id: test
273-
run: |
274-
# Tests use parts of ProofWidgets not imported by Mathlib.
275-
# Ensure everything has been built.
276-
lake build ProofWidgets
277-
make -j 8 test
272+
uses: liskin/gh-problem-matcher-wrap@v2
273+
with:
274+
linters: gcc
275+
run: |
276+
bash -o pipefail -c "
277+
# Tests use parts of ProofWidgets not imported by Mathlib.
278+
# Ensure everything has been built.
279+
lake build ProofWidgets
280+
make -j 8 test"
278281
279282
- name: lint mathlib
280283
id: lint

.github/workflows/build.yml.in

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -248,12 +248,15 @@ jobs:
248248
lake exe shake
249249

250250
- name: test mathlib
251-
id: test
252-
run: |
253-
# Tests use parts of ProofWidgets not imported by Mathlib.
254-
# Ensure everything has been built.
255-
lake build ProofWidgets
256-
make -j 8 test
251+
uses: liskin/gh-problem-matcher-wrap@v2
252+
with:
253+
linters: gcc
254+
run: |
255+
bash -o pipefail -c "
256+
# Tests use parts of ProofWidgets not imported by Mathlib.
257+
# Ensure everything has been built.
258+
lake build ProofWidgets
259+
make -j 8 test"
257260

258261
- name: lint mathlib
259262
id: lint

.github/workflows/build_fork.yml

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -266,12 +266,15 @@ jobs:
266266
lake exe shake
267267
268268
- name: test mathlib
269-
id: test
270-
run: |
271-
# Tests use parts of ProofWidgets not imported by Mathlib.
272-
# Ensure everything has been built.
273-
lake build ProofWidgets
274-
make -j 8 test
269+
uses: liskin/gh-problem-matcher-wrap@v2
270+
with:
271+
linters: gcc
272+
run: |
273+
bash -o pipefail -c "
274+
# Tests use parts of ProofWidgets not imported by Mathlib.
275+
# Ensure everything has been built.
276+
lake build ProofWidgets
277+
make -j 8 test"
275278
276279
- name: lint mathlib
277280
id: lint

0 commit comments

Comments
 (0)