Izaron marked 2 inline comments as done. Izaron added inline comments.
================ Comment at: clang-tools-extra/test/clang-tidy/checkers/modernize/use-inline-const-variables-in-headers.hpp:1 +// RUN: %check_clang_tidy %s -std=c++17 modernize-use-inline-const-variables-in-headers %t + ---------------- carlosgalvezp wrote: > Should tests be added for the `CheckNonInline` and `CheckExtern` options? I decided to split the test into two files as it is done for other checkers that have multiple options. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118743/new/ https://reviews.llvm.org/D118743 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
