Skip to content

Commit

Permalink
Update C++ linter settings to speed up cppcheck
Browse files Browse the repository at this point in the history
Needs AMReX-Astro/cpp-linter-action#8 to be merged. I've currently got
it pointing to my feature branch, so we can make sure it works properly.
  • Loading branch information
yut23 committed Nov 30, 2023
1 parent f104eff commit 19faf50
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions .github/workflows/c-linter.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,13 +28,15 @@ jobs:
sudo apt-get -qq -y install curl clang-tidy cmake jq clang cppcheck clang-format bear g++>=9.3.0
- name: Run cpp linter
uses: AMReX-Astro/cpp-linter-action@main
uses: yut23/cpp-linter-action@cppcheck-improvements
with:
build_path: 'unit_test/test_react'
make_options: '-j 2 USE_OMP=FALSE USE_MPI=FALSE USE_CUDA=FALSE DEBUG=TRUE'
ignore_files: 'amrex'
ignore_files: 'amrex|util/gcem'
header_filter: '(/conductivity/|/constants/|/EOS/|/integration/|/interfaces/|/networks/|/neutrinos/|/nse_solver/|/opacity/|/rates/|/screening/|/util/|^\./).*\.H$'
config_file: ${GITHUB_WORKSPACE}/.clang-tidy
# not sure why cppcheck tests _MSC_VER, since it's only referenced from gcem
cppcheck_options: '-U_OPENMP -UAMREX_USE_GPU -UAMREX_USE_CUDA -UAMREX_USE_HIP -U_MSC_VER'

- name: Archive clang tidy report
uses: actions/upload-artifact@v1
Expand Down

0 comments on commit 19faf50

Please sign in to comment.