Skip to content

Commit

Permalink
clang-tidy: Raise minimum clang-tidy version to 15
Browse files Browse the repository at this point in the history
  • Loading branch information
edalm committed Feb 6, 2025
1 parent 11263eb commit 0f6425d
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
4 changes: 2 additions & 2 deletions build-support/macros-and-definitions.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -259,8 +259,8 @@ macro(process_options)

if(${NS3_CLANG_TIDY})
find_program(
CLANG_TIDY NAMES clang-tidy clang-tidy-14 clang-tidy-15 clang-tidy-16
clang-tidy-17 clang-tidy-18
CLANG_TIDY NAMES clang-tidy clang-tidy-15 clang-tidy-16 clang-tidy-17
clang-tidy-18
)
if("${CLANG_TIDY}" STREQUAL "CLANG_TIDY-NOTFOUND")
message(FATAL_ERROR "Clang-tidy was not found")
Expand Down
5 changes: 3 additions & 2 deletions doc/contributing/source/coding-style.rst
Original file line number Diff line number Diff line change
Expand Up @@ -237,9 +237,10 @@ Therefore, it is recommended to use the latest version available.

To ensure consistency among developers, |ns3| defines a minimum version of clang-tidy,
whose warnings must not be ignored. Therefore, developers should, at least, scan their
code with the minimum version of clang-tidy.
code with the minimum version of clang-tidy. However, more recent versions can be used,
which will produce better warnings.

The minimum version is clang-tidy-14.
The minimum version is clang-tidy-15.

Integration with IDEs
=====================
Expand Down

0 comments on commit 0f6425d

Please sign in to comment.