added doxygen to workflow

Signed-off-by: Peter Geurts <[email protected]>

.github/workflows/linting.yml

@@ -37,9 +37,16 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - run: sudo apt install -y clang-format
+      - run: sudo apt update && sudo apt install -y clang-format
       - run: find ros2_ws/src -iname '*.h' -o -iname '*.hpp' -o -iname '*.cpp' | xargs clang-format --dry-run --Werror
 
+  doxygen:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - run: sudo apt update && sudo apt install -y doxygen
+      - run: ./doxygen-lint.sh 
+
   reuse:
     runs-on: ubuntu-latest
     steps:

doxygen-lint.sh

@@ -3,8 +3,20 @@
 # SPDX-FileCopyrightText: Alliander N. V.
 # 
 # SPDX-License-Identifier: Apache-2.0
-OUTPUT=$(doxygen doxyfile.lint /dev/null 2>&1)
-NUM_WARNINGS=$(($(echo "$OUTPUT" | wc -l) - 1))
+FILENAME=doxyfile.lint
+
+if ! command -v doxygen > /dev/null 2>&1 ; then
+  echo "Doxygen not found, exiting."
+  exit 1
+fi
+
+if [ ! -f $FILENAME ]; then
+  echo "Doxyfile '$FILENAME' not found, exiting."
+  exit 1
+fi
+
+OUTPUT=$(doxygen $FILENAME /dev/null 2>&1)
+NUM_WARNINGS=$(echo "$OUTPUT" | grep -i "warning:" | wc -l)
 
 if [ "$NUM_WARNINGS" -gt 0 ]; then
   echo "$OUTPUT"

ros2_ws/src/rcdt_moveit/include/moveit_manager.hpp

@@ -47,7 +47,6 @@ class MoveitManager {
   /**
    * constructor for the MoveitManager class.
    * @param node The ROS2 node to attach to.
-   * @return An instance of MoveitManager.
    */
   MoveitManager(rclcpp::Node::SharedPtr node);