Skip to content

Commit e8f1dcc

Browse files
fix clang comments
Signed-off-by: AdityaPandeyCN <[email protected]>
1 parent 9028ee7 commit e8f1dcc

File tree

1 file changed

+7
-12
lines changed

1 file changed

+7
-12
lines changed

.github/workflows/clang-tidy-review-post.yml

Lines changed: 7 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -11,26 +11,21 @@ permissions:
1111
pull-requests: write
1212

1313
concurrency:
14-
group: ${{ github.workflow }}-${{ github.event.workflow_run.pull_requests[0].number }}
14+
group: ${{ github.workflow }}-${{ github.event.pull_request.number }}
1515
cancel-in-progress: true
1616

1717
jobs:
18-
post-comments:
19-
# Only run if the triggering workflow was from a pull request
20-
if: github.event.workflow_run.event == 'pull_request'
18+
build:
2119
runs-on: ubuntu-latest
22-
20+
2321
steps:
2422
- name: Post review comments
2523
id: post-review
2624
uses: ZedThree/clang-tidy-review/[email protected]
2725
with:
2826
max_comments: 10
29-
30-
# Fail if there are any clang-tidy warnings
31-
- name: Check for issues
32-
if: steps.post-review.outputs.total_comments > 0
33-
run: |
34-
echo "::error::Found ${{ steps.post-review.outputs.total_comments }} clang-tidy issues"
35-
exit 1
27+
28+
# If there are any comments, fail the check
29+
- if: steps.post-review.outputs.total_comments > 0
30+
run: exit 1
3631

0 commit comments

Comments
 (0)