Skip to content

Commit

Permalink
ci: Run clang-tidy on the ActSVG plugin
Browse files Browse the repository at this point in the history
In acts-project#3634, I am struggling with clang-tidy errors which popped up after I
had to transiently enable the ActSVG plugin. In order to solve this
problem is propose we unconditionally enable clang-tidy on ActSVG. This
PR does that, and it also fixes any existing clang-tidy issues.
  • Loading branch information
stephenswat committed Sep 23, 2024
1 parent cb124e8 commit 44380b8
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 20 deletions.
1 change: 0 additions & 1 deletion CMakePresets.json
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,6 @@
"displayName": "GitLab-CI",
"inherits": "ci-common",
"cacheVariables": {
"ACTS_BUILD_PLUGIN_ACTSVG": "OFF",
"ACTS_BUILD_ODD": "OFF",
"ACTS_BUILD_EXAMPLES_PYTHON_BINDINGS": "OFF",
"ACTS_RUN_CLANG_TIDY": "ON"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,7 @@
#include "Acts/Utilities/Logger.hpp"
#include <actsvg/core.hpp>

namespace Acts {

namespace Svg {

namespace EventDataConverter {
namespace Acts::Svg::EventDataConverter {

/// Write/create a 3D point in XY view
///
Expand Down Expand Up @@ -62,8 +58,4 @@ actsvg::svg::object point(const Vector3& pos, ActsScalar size,
return circle;
}

} // namespace EventDataConverter

} // namespace Svg

} // namespace Acts
} // namespace Acts::Svg::EventDataConverter
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,7 @@
#include <tuple>
#include <vector>

namespace Acts {

namespace Svg {
namespace Acts::Svg {

using ProtoSurface = actsvg::proto::surface<std::vector<Vector3>>;
using ProtoGrid = actsvg::proto::grid;
Expand Down Expand Up @@ -340,5 +338,4 @@ static inline actsvg::svg::object zphi(
}

} // namespace View
} // namespace Svg
} // namespace Acts
} // namespace Acts::Svg
6 changes: 2 additions & 4 deletions Plugins/ActSVG/include/Acts/Plugins/ActSVG/SvgUtils.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,7 @@
#include <tuple>
#include <vector>

namespace Acts {
namespace Svg {
namespace Acts::Svg {

struct Style {
// Fill parameters
Expand Down Expand Up @@ -178,5 +177,4 @@ inline static void toFile(const std::vector<actsvg::svg::object>& objects,
fout.close();
}

} // namespace Svg
} // namespace Acts
} // namespace Acts::Svg

0 comments on commit 44380b8

Please sign in to comment.