CI Update (#20)

* Updated clang format job

* Updated Ubuntu CI job to run on focal, jammy, and noble

* Update to non-deprecated version of upload artifact action

* Changed name of variable per clang-tidy

Author: marip8

.github/workflows/clang_format.yml

@@ -10,15 +10,16 @@ on:
 
 jobs:
   clang_format:
-    name: Clang-Format
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v1
+      - uses: actions/checkout@v4
 
       - name: Run clang format
         run: |
           sudo apt update
           sudo apt install -y git clang-format
+          if [ $? -ge 1 ]; then return 1; fi
           ./.run-clang-format
+          if [ $? -ge 1 ]; then return 1; fi
           output=$(git diff)
           if [ -n "$output" ]; then exit 1; else exit 0; fi

.github/workflows/ubuntu.yml

@@ -0,0 +1,43 @@
+name: Ubuntu [Focal, Jammy, Noble]
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+  schedule:
+    - cron: '0 5 * * *'
+  workflow_dispatch:
+  release:
+    types:
+      - released
+
+jobs:
+  ci:
+    name: ${{ matrix.distro }}
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        distro: [focal, jammy, noble]
+    container:
+      image: ubuntu:${{ matrix.distro }}
+      env:
+        CCACHE_DIR: ${{ github.workspace }}/${{ matrix.distro }}/.ccache
+        DEBIAN_FRONTEND: noninteractive
+        TZ: Etc/UTC
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+        with:
+          path: target_ws/src
+
+      - name: Build and Tests
+        uses: tesseract-robotics/colcon-action@v9
+        with:
+          before-script: 'apt install -y -qq clang-tidy lcov'
+          ccache-prefix: ${{ matrix.distro }}
+          add-ros-ppa: true
+          vcs-file: dependencies.repos
+          target-path: target_ws/src
+          target-args: --cmake-args -DCMAKE_BUILD_TYPE=Debug -DBUILD_TESTING=ON -DENABLE_CLANG_TIDY=ON -DENABLE_CODE_COVERAGE=ON

.github/workflows/ubuntu_focal.yml

@@ -87,13 +87,13 @@ jobs:
         cp ./*.nupkg ${{ github.workspace }}/artifacts
         cp ./*.tar.xz ${{ github.workspace }}/artifacts
 
-    - uses: actions/upload-artifact@v2
+    - uses: actions/upload-artifact@v4
       if: ${{ github.event_name == 'release' && github.event.action == 'released' }}
       with:
         name: nuget_package
         path: ${{ github.workspace }}/artifacts/*.nupkg
 
-    - uses: actions/upload-artifact@v2
+    - uses: actions/upload-artifact@v4
       if: ${{ github.event_name == 'release' && github.event.action == 'released' }}
       with:
         name: archive_package

.github/workflows/windows_2019.yml

@@ -110,17 +110,17 @@ std::vector<std::string> getAllAvailableSections(const std::string& library_name
 
 std::string decorate(const std::string& library_name, const std::string& library_directory)
 {
-  boost::filesystem::path sl;
+  boost::filesystem::path lib_path;
   if (library_directory.empty())
-    sl = boost::filesystem::path(library_name);
+    lib_path = boost::filesystem::path(library_name);
   else
-    sl = boost::filesystem::path(library_directory) / library_name;
+    lib_path = boost::filesystem::path(library_directory) / library_name;
 
   boost::filesystem::path actual_path =
-      (std::strncmp(sl.filename().string().c_str(), "lib", 3) != 0 ?
-           boost::filesystem::path((sl.has_parent_path() ? sl.parent_path() / L"lib" : L"lib").native() +
-                                   sl.filename().native()) :
-           sl);
+      (std::strncmp(lib_path.filename().string().c_str(), "lib", 3) != 0 ?
+           boost::filesystem::path((lib_path.has_parent_path() ? lib_path.parent_path() / L"lib" : L"lib").native() +
+                                   lib_path.filename().native()) :
+           lib_path);
 
   actual_path += boost::dll::shared_library::suffix();
   return actual_path.string();