fix sonar

Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>

Author: mgovers

power_grid_model_c/power_grid_model/include/power_grid_model/common/common.hpp

@@ -92,8 +92,9 @@ template <class T, class... Ts>
 concept is_in_list_c = (std::same_as<std::remove_const_t<T>, Ts> || ...);
 
 namespace capturing {
-// NOLINTNEXTLINE(cppcoreguidelines-missing-std-forward) // perfect forward into void
-template <class... T> constexpr void into_the_void(T&&... /*ignored*/) {
+// perfect forward into void
+template <class... T>
+constexpr void into_the_void(T&&... /*ignored*/) { // NOLINT(cppcoreguidelines-missing-std-forward)
     // do nothing; the constexpr allows all compilers to optimize this away
 }
 } // namespace capturing

power_grid_model_c/power_grid_model/include/power_grid_model/main_model_impl.hpp

@@ -522,9 +522,9 @@ class MainModelImpl<ExtraRetrievableTypes<ExtraRetrievableType...>, ComponentLis
                                *meta_data_,
                            },
                            ignore_output);
-        } catch (const SparseMatrixError&) { // NOLINT(bugprone-empty-catch)
+        } catch (const SparseMatrixError&) { // NOLINT(bugprone-empty-catch) // NOSONAR
             // missing entries are provided in the update data
-        } catch (const NotObservableError&) { // NOLINT(bugprone-empty-catch)
+        } catch (const NotObservableError&) { // NOLINT(bugprone-empty-catch) // NOSONAR
             // missing entries are provided in the update data
         }
 

tests/cpp_unit_tests/test_common.cpp

@@ -18,7 +18,7 @@ static_assert(std::invocable<IncludeAll, Idx>);
 static_assert(std::invocable<IncludeAll, Idx, Idx>);
 static_assert(include_all());
 static_assert(include_all(1));
-static_assert(include_all(Idx{2},
-                          std::move(Idx{3}))); // NOLINT(performance-move-const-arg) // to test that rvalues work
+static_assert(include_all(
+    Idx{2}, std::move(Idx{3}))); // NOLINT(performance-move-const-arg) // NOSONAR // to test that rvalues work
 } // namespace
 } // namespace power_grid_model

tests/cpp_unit_tests/test_math_solver_common.hpp

@@ -112,7 +112,8 @@ template <symmetry_tag sym_type> struct SteadyStateSolverTestGrid {
     static constexpr double v1 = 0.97;
     static constexpr double v2 = 0.90;
     static constexpr double deg = deg_30 / 30.0;
-    // NOLINTBEGIN(cppcoreguidelines-avoid-const-or-ref-data-members) // should be constexpr but cant due to std::exp
+    // NOLINTBEGIN(cppcoreguidelines-avoid-const-or-ref-data-members) // NOSONAR // should be constexpr but cant due to
+    // std::exp
     DoubleComplex const u0 = v0 * std::exp(-1.0i * deg);
     DoubleComplex const u1 = v1 * std::exp(-4.0i * deg);
     DoubleComplex const u2 = v2 * std::exp(-37.0i * deg);
@@ -137,7 +138,7 @@ template <symmetry_tag sym_type> struct SteadyStateSolverTestGrid {
     // injection of shunt0 at bus2
     DoubleComplex const i2_shunt_inj = branch1_i_t;
     DoubleComplex const ys = -i2_shunt_inj / u2; // output
-    // NOLINTEND(cppcoreguidelines-avoid-const-or-ref-data-members)
+    // NOLINTEND(cppcoreguidelines-avoid-const-or-ref-data-members) // NOSONAR
 
     SolverOutput<sym> output_ref() const {
         if constexpr (is_symmetric_v<sym>) {