OilfieldToolsNet/ResInsight
4
0
Fork 0
mirror of https://github.com/OPM/ResInsight.git synced 2025-02-25 18:55:39 -06:00
Code Issues Packages Projects Releases Wiki Activity

Use clang-tidy-15

Browse Source 
Update clang-tidy.yml to use clang-tidy-15 to be able to support c++20

Do not use OpenMP, as omp.h is not found during parsing

Remove modernize-use-nullptr, as this does not work with <=> (spaceship operator)
https://reviews.llvm.org/D95714
This commit is contained in:
Magne Sjaastad 2023-03-21 10:02:03 +01:00 committed by GitHub
parent 535811cc4f
commit 706c8a5266
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 11 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
.github/workflows/clang-tidy.yml vendored
View File

@ -9,13 +9,14 @@ on:
jobs:
ResInsight-x64-buildcache:
runs-on: ${{ matrix.config.os }}
continue-on-error: true
strategy:
fail-fast: false
matrix:
config:
- {
name: "Ubuntu 20.04",
os: ubuntu-20.04,
name: "Ubuntu 22.04",
os: ubuntu-22.04,
cc: "gcc",
cxx: "g++",
vcpkg-response-file: vcpkg_x64-linux.txt,
@ -60,30 +61,30 @@ jobs:
run: |
mkdir build
cd build
cmake -DCMAKE_TOOLCHAIN_FILE=${{ matrix.config.cmake-toolchain }} -DCMAKE_EXPORT_COMPILE_COMMANDS=ON ..
sudo apt-get install clang-tidy
cmake -DCMAKE_TOOLCHAIN_FILE=${{ matrix.config.cmake-toolchain }} -DRESINSIGHT_USE_OPENMP=OFF -DCMAKE_EXPORT_COMPILE_COMMANDS=ON ..
sudo apt-get install clang-tidy-15
mv compile_commands.json compile_commands_original.json
python ../scripts/fix_json_database.py compile_commands_original.json >> compile_commands.json
- name: clang-tidy on ApplicationLibCode/Application
run: |
cd build
run-clang-tidy -fix files ApplicationLibCode/Application
run-clang-tidy-15 -fix files ApplicationLibCode/Application
- name: clang-tidy on ApplicationLibCode/ProjectDataModel
run: |
cd build
run-clang-tidy -fix files ApplicationLibCode/ProjectDataModel
run-clang-tidy-15 -fix files ApplicationLibCode/ProjectDataModel
- name: clang-tidy on ApplicationLibCode/ReservoirDataModel
run: |
cd build
run-clang-tidy -fix files ApplicationLibCode/ReservoirDataModel
run-clang-tidy-15 -fix files ApplicationLibCode/ReservoirDataModel
- name: clang-tidy on ApplicationLibCode/Commands
run: |
cd build
run-clang-tidy -fix files ApplicationLibCode/Commands
run-clang-tidy-15 -fix files ApplicationLibCode/Commands
- name: clang-tidy on ApplicationLibCode/GeoMech
run: |
cd build
run-clang-tidy -fix files ApplicationLibCode/GeoMech
run-clang-tidy-15 -fix files ApplicationLibCode/GeoMech
- name: Remove Qt before creating PR
run: |
rm -rf Qt

2
ApplicationLibCode/.clang-tidy
View File

@ -1,5 +1,5 @@
---
Checks: '-*,modernize-use-override,modernize-use-nullptr,modernize-deprecated-headers,bugprone-bool-pointer-implicit-conversion,bugprone-parent-virtual-call,bugprone-redundant-branch-condition,bugprone-suspicious-semicolon,bugprone-suspicious-string-compare'
Checks: '-*,modernize-use-override,modernize-deprecated-headers,bugprone-bool-pointer-implicit-conversion,bugprone-parent-virtual-call,bugprone-redundant-branch-condition,bugprone-suspicious-semicolon,bugprone-suspicious-string-compare'
WarningsAsErrors: ''
HeaderFilterRegex: 'ApplicationLibCode/*.*$'
FormatStyle: 'file'