Skip to content

Commit 8a9d879

Browse files
committed
Update cpp-linter action.
- Replace permissions from 'write' to 'read' to creat unified experience. PRs created from forks are not allowed to create PR comments, so setting 'write' permission works only for PR created from the local branches. - Updated versions to the latest release versions. - Use clang-format from 21 release. - Use step-summary to report clang-format results instead of PR comments.
1 parent c0c27d3 commit 8a9d879

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

.github/workflows/cts_ci.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -89,20 +89,20 @@ jobs:
8989

9090
runs-on: ubuntu-latest
9191
permissions:
92-
pull-requests: write
92+
contents: read
93+
pull-requests: read
9394
steps:
94-
- uses: actions/checkout@v4
95-
- uses: cpp-linter/cpp-linter-action@main
95+
- uses: actions/checkout@v5
96+
- uses: cpp-linter/cpp-linter-action@v2
9697
id: linter
97-
continue-on-error: true
9898
env:
9999
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
100100
with:
101101
style: file
102+
version: '21'
102103
tidy-checks: '-*' # Disable all clang-tidy checks for now. Enable checks as needed.
104+
step-summary: true
103105
lines-changed-only: true
104-
format-review: true
105-
passive-reviews: true
106106

107107
compile-cts:
108108
needs: build-image-for-sycl-impl

0 commit comments

Comments
 (0)