Skip to content

Commit 86e2b66

Browse files
mitsudome-rxmfcx
authored andcommitted
Revert "ci(build-test-tidy): introduce success condition jobs (autowarefoundation#9769)"
This reverts commit 421ec7d.
1 parent c8e0040 commit 86e2b66

File tree

2 files changed

+0
-106
lines changed

2 files changed

+0
-106
lines changed

.github/actions/evaluate-job-result/action.yaml

-45
This file was deleted.

.github/workflows/build-test-tidy-pr.yaml

-61
Original file line numberDiff line numberDiff line change
@@ -58,34 +58,6 @@ jobs:
5858
secrets:
5959
codecov-token: ${{ secrets.CODECOV_TOKEN }}
6060

61-
build-test-pr:
62-
needs:
63-
- build-and-test-differential
64-
- build-and-test-differential-cuda
65-
if: ${{ always() }}
66-
runs-on: ubuntu-latest
67-
steps:
68-
- uses: actions/checkout@v4
69-
70-
- name: Initialize Summary
71-
run: echo "### Build Test PR Results" > $GITHUB_STEP_SUMMARY
72-
shell: bash
73-
74-
- name: Evaluate build-and-test-differential
75-
uses: ./.github/actions/evaluate-job-result
76-
with:
77-
job_result: ${{ needs.build-and-test-differential.result }}
78-
job_name: build-and-test-differential
79-
expected_results: success
80-
81-
- name: Evaluate build-and-test-differential-cuda
82-
if: ${{ always() }}
83-
uses: ./.github/actions/evaluate-job-result
84-
with:
85-
job_result: ${{ needs.build-and-test-differential-cuda.result }}
86-
job_name: build-and-test-differential-cuda
87-
expected_results: success,skipped
88-
8961
clang-tidy-differential:
9062
needs:
9163
- check-if-cuda-job-is-needed
@@ -102,36 +74,3 @@ jobs:
10274
with:
10375
container: ghcr.io/autowarefoundation/autoware:universe-devel
10476
container-suffix: -cuda
105-
106-
clang-tidy-pr:
107-
needs:
108-
- clang-tidy-differential
109-
- clang-tidy-differential-cuda
110-
if: ${{ always() }}
111-
runs-on: ubuntu-latest
112-
steps:
113-
- name: Initialize Summary
114-
run: echo "### Clang Tidy PR Results" > $GITHUB_STEP_SUMMARY
115-
shell: bash
116-
117-
- name: Check clang-tidy success
118-
if: ${{ needs.clang-tidy-differential.result == 'success' || needs.clang-tidy-differential-cuda.result == 'success' }}
119-
run: |
120-
echo "✅ Either one of the following has succeeded:" >> $GITHUB_STEP_SUMMARY
121-
122-
- name: Fail if conditions not met
123-
if: ${{ !(needs.clang-tidy-differential.result == 'success' || needs.clang-tidy-differential-cuda.result == 'success') }}
124-
run: |
125-
echo "::error::❌ Either one of the following should have succeeded:"
126-
echo "::error::clang-tidy-differential: ${{ needs.clang-tidy-differential.result }}"
127-
echo "::error::clang-tidy-differential-cuda: ${{ needs.clang-tidy-differential-cuda.result }}"
128-
129-
echo "❌ Either one of the following should have succeeded:" >> $GITHUB_STEP_SUMMARY
130-
131-
exit 1
132-
133-
- name: Print the results
134-
if: ${{ always() }}
135-
run: |
136-
echo "- **clang-tidy-differential:** ${{ needs.clang-tidy-differential.result }}" >> $GITHUB_STEP_SUMMARY
137-
echo "- **clang-tidy-differential-cuda:** ${{ needs.clang-tidy-differential-cuda.result }}" >> $GITHUB_STEP_SUMMARY

0 commit comments

Comments
 (0)