Merge topic 'ci-clazy'

bdb9d818d2 ci: Add lint job to run clazy on our C++ code
c6b2439aad ci: Add clazy to Fedora base image
a1b7710241 ci: configure 'typos' to tolerate a standard streambuf member
24c7c13cc6 Source: Fix typos
564f13fed6 Merge branch 'lfortran-module-mangling' into ci-clazy

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !10966
This commit is contained in:
Brad King
2025-07-28 13:43:22 +00:00
committed by Kitware Robot
7 changed files with 51 additions and 2 deletions
+7
View File
@@ -140,6 +140,13 @@ l:clang-analyzer-fedora42:
variables:
CMAKE_CI_JOB_NIGHTLY: "true"
l:clazy-fedora42:
extends:
- .fedora42_clazy
- .cmake_build_linux
- .linux_x86_64_tags
- .run_automatically
# Linux builds
b:centos7-x86_64:
@@ -0,0 +1,4 @@
set(configure_no_sccache 1)
set(CMake_QT_MAJOR_VERSION "5" CACHE STRING "")
include("${CMAKE_CURRENT_LIST_DIR}/configure_fedora42_common.cmake")
@@ -38,6 +38,7 @@ texinfo
# Install lint tools.
clang-analyzer
clazy
codespell
# Tools needed for the test suite.
+26
View File
@@ -0,0 +1,26 @@
export CC=/usr/bin/clang
export CXX=/usr/bin/clazy
export CLANGXX=/usr/bin/clang++
export CLAZY_CHECKS="level2\
,no-base-class-event\
,no-connect-3arg-lambda\
,no-connect-by-name\
,no-container-inside-loop\
,no-copyable-polymorphic\
,no-ctor-missing-parent-argument\
,no-function-args-by-ref\
,no-function-args-by-value\
,no-missing-qobject-macro\
,no-non-pod-global-static\
,no-old-style-connect\
,no-qproperty-without-notify\
,no-qstring-allocations\
,no-range-loop-detach\
,no-range-loop-reference\
,no-reserve-candidates\
,no-rule-of-three\
,no-rule-of-two-soft\
,no-static-pmf\
,no-strict-iterators\
"
+10 -1
View File
@@ -83,7 +83,7 @@
### Fedora
.fedora42:
image: "kitware/cmake:ci-fedora42-x86_64-2025-04-22"
image: "kitware/cmake:ci-fedora42-x86_64-2025-07-22"
variables:
GIT_CLONE_PATH: "$CI_BUILDS_DIR/cmake ci/long file name for testing purposes"
@@ -115,6 +115,15 @@
CTEST_NO_WARNINGS_ALLOWED: 1
CMAKE_CI_NO_INSTALL: 1
.fedora42_clazy:
extends: .fedora42
variables:
CMAKE_CONFIGURATION: fedora42_clazy
CMAKE_CI_BUILD_TYPE: Debug
CTEST_NO_WARNINGS_ALLOWED: 1
CMAKE_CI_NO_INSTALL: 1
.fedora42_sphinx:
extends: .fedora42
+2
View File
@@ -70,6 +70,8 @@ nto = "nto"
[type.cpp.extend-identifiers]
APPENDed = "APPENDed"
setp = "setp"
setp_ = "setp_"
[type.json.extend-identifiers]
# Some compiler options from `Templates/MSBuild/FlagTables/*.json` trigger too many false-positives.
@@ -303,7 +303,7 @@ std::map<std::string, std::string> GetOSReleaseVariables(
scripts = gl.GetFiles();
}
// 2. User provided (append to the CMake prvided)
// 2. User provided (append to the CMake provided)
cmList::append(
scripts, makefile.GetDefinition("CMAKE_GET_OS_RELEASE_FALLBACK_SCRIPTS"));