Skip to content

Commit 6362d8e

Browse files
committed
CI: Reorder steps
1 parent 54579ca commit 6362d8e

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

.github/workflows/ci.yaml

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -75,31 +75,31 @@ jobs:
7575
uses: rhaschke/industrial_ci@clang-tidy
7676
env: ${{ matrix.env }}
7777

78-
- name: Show clang-tidy warnings
79-
if: always() && matrix.env.CLANG_TIDY
80-
uses: asarium/clang-tidy-action@v1
81-
with:
82-
fixesFile: ${{ env.BASEDIR }}/clang-tidy-fixes.yaml
83-
8478
- name: Upload ici's target_ws/install folder
8579
uses: rhaschke/upload-ici-workspace@main
8680
if: success() && matrix.env.CLANG_TIDY
8781
with:
8882
subdir: target_ws/install
8983

84+
- name: Upload test artifacts (on failure)
85+
uses: actions/upload-artifact@v3
86+
if: failure() && (steps.ici.outputs.run_target_test || steps.ici.outputs.target_test_results)
87+
with:
88+
name: test-results-${{ matrix.env.IMAGE }}${{ matrix.env.NAME && '-' || ''}}${{ matrix.env.NAME }}
89+
path: ${{ env.BASEDIR }}/target_ws/**/test_results/**/*.xml
90+
9091
- name: Upload clang-tidy fixes (on failure)
9192
uses: actions/upload-artifact@v3
92-
if: failure() && (steps.ici.outputs.clang_tidy_checks)
93+
if: failure() && steps.ici.outputs.clang_tidy_checks
9394
with:
9495
name: clang-tidy-fixes.yaml
9596
path: ${{ env.BASEDIR }}/clang-tidy-fixes.yaml
9697

97-
- name: Upload test artifacts (on failure)
98-
uses: actions/upload-artifact@v3
99-
if: failure() && (steps.ici.outputs.run_target_test || steps.ici.outputs.target_test_results)
98+
- name: Show clang-tidy warnings
99+
if: always() && matrix.env.CLANG_TIDY
100+
uses: asarium/clang-tidy-action@v1
100101
with:
101-
name: test-results-${{ matrix.env.IMAGE }}${{ matrix.env.NAME && '-' || ''}}${{ matrix.env.NAME }}
102-
path: ${{ env.BASEDIR }}/target_ws/**/test_results/**/*.xml
102+
fixesFile: ${{ env.BASEDIR }}/clang-tidy-fixes.yaml
103103

104104
- name: Generate codecov report
105105
uses: rhaschke/lcov-action@main

0 commit comments

Comments
 (0)