Skip to content

Commit

Permalink
need screenshot of step summary
Browse files Browse the repository at this point in the history
  • Loading branch information
2bndy5 committed Dec 23, 2024
1 parent 88d5db6 commit 9c83a2b
Showing 1 changed file with 15 additions and 13 deletions.
28 changes: 15 additions & 13 deletions .github/workflows/cpp-lint-package.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,15 +11,16 @@ on:

jobs:
cpp-linter:
name: cpp-linter job
runs-on: windows-latest

strategy:
matrix:
clang-version: ['7', '8', '9','10', '11', '12', '13', '14', '15', '16', '17']
repo: ['cpp-linter/cpp-linter']
branch:
- ${{ github.event_name == 'workflow_dispatch' && inputs.branch || 'hide-reuse-pr-reviews' }}
fail-fast: false
# strategy:
# matrix:
# clang-version: ['17'] #['7', '8', '9','10', '11', '12', '13', '14', '15', '16', '17']
# repo: ['cpp-linter/cpp-linter']
# branch:
# - ${{ github.event_name == 'workflow_dispatch' && inputs.branch || 'hide-reuse-pr-reviews' }}
# fail-fast: false

steps:

Expand All @@ -32,10 +33,10 @@ jobs:
python-version: 3.x

- name: Install workflow deps
run: python -m pip install clang-tools git+https://github.com/${{ matrix.repo }}/@${{ matrix.branch }}
run: python -m pip install clang-tools git+https://github.com/cpp-linter/cpp-linter/

- name: Install clang-tools
run: clang-tools --install ${{ matrix.clang-version }} --directory ${{ runner.temp }}/llvm
run: clang-tools --install 17 --directory ${{ runner.temp }}/llvm

- name: Setup VS dev env
uses: TheMrMilchmann/setup-msvc-dev@v3
Expand Down Expand Up @@ -66,10 +67,11 @@ jobs:
-V=${{ runner.temp }}/llvm
-f=false
--extra-arg="-std=c++14 -Wall"
--thread-comments=${{ matrix.clang-version == '17' && 'update' }}
-a=${{ matrix.clang-version == '17' }}
--tidy-review=${{ matrix.clang-version == '17' }}
--format-review=${{ matrix.clang-version == '17' }}
--thread-comments=update
-a=false
--tidy-review=17
--format-review=17
--step-summary=true
- name: Fail fast?!
if: steps.linter.outputs.checks-failed > 0
Expand Down

0 comments on commit 9c83a2b

Please sign in to comment.