CI update: Jammy ROS One, clang-format-12

ros-visualization · Nov 4, 2023 · 13d6d6f · 13d6d6f
1 parent e03e7c8
commit 13d6d6f
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 12 deletions.
diff --git a/.github/workflows/Dockerfile b/.github/workflows/Dockerfile
@@ -1,9 +1,12 @@
 # rhaschke/ici:rviz-noetic-ros
 # Image used for GHA CI builds
-# build with: DOCKER_BUILDKIT=1 docker build --tag rhaschke/ici:rviz-noetic-ros - < .github/workflows/Dockerfile
+# build with:
+# - DOCKER_BUILDKIT=1 docker build --tag rhaschke/ici:rviz-noetic-ros - < .github/workflows/Dockerfile
+# - DOCKER_BUILDKIT=1 docker build --tag rhaschke/ici:rviz-jammy-ros --build-arg="BASE=ubiagni/ros:jammy-ros-base" --build-arg="ROS_DISTRO=one" - < .github/workflows/Dockerfile
 
 ARG ROS_DISTRO=noetic
-FROM ros:${ROS_DISTRO}-ros-base
+ARG BASE=ros:noetic-ros-base
+FROM ${BASE}
 
 ENV TERM xterm
 
@@ -19,11 +22,9 @@ RUN --mount=type=bind,target=/tmp/rviz \
         # Some basic requirements
         wget git sudo \
         # Preferred build tools
-        python$(test "${ROS_DISTRO}" = "noetic" && echo 3)-catkin-tools \
-        clang clang-format-10 clang-tidy clang-tools \
-        ccache && \
+        clang clang-format-12 clang-tidy clang-tools ccache && \
     #
-    # Download all dependencies of MoveIt
+    # Download all dependencies for rviz
     rosdep update $(test "${ROS_DISTRO}" = "melodic" && echo "--include-eol-distros") && \
     DEBIAN_FRONTEND=noninteractive \
     rosdep install -y --from-paths /tmp/rviz --ignore-src --rosdistro ${ROS_DISTRO} --as-root=apt:false && \

diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -13,7 +13,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        distro: [noetic]
+        distro: [noetic, jammy]
         ogre: ["1.9", "1.12"]
         include:
           - distro: noetic
@@ -28,7 +28,7 @@ jobs:
       CATKIN_LINT: true
       CCACHE_DIR: ${{ github.workspace }}/.ccache
       BASEDIR: /home/runner/work
-      DOCKER_IMAGE: rhaschke/ici:rviz-${{ matrix.distro }}-${{ matrix.repo || 'ros' }}
+      DOCKER_IMAGE: rhaschke/ici:rviz-${{ matrix.distro }}-ros
       CACHE_PREFIX: ${{ matrix.distro }}
       # perform full clang-tidy check only on manual trigger (workflow_dispatch), PRs do check changed files, otherwise nothing
       CLANG_TIDY_BASE_REF: ${{ github.event_name != 'workflow_dispatch' && (github.base_ref || github.ref) || '' }}
@@ -51,7 +51,6 @@ jobs:
 
       - name: industrial_ci
         uses: rhaschke/industrial_ci@master
-        env: ${{ matrix.env || env }}
 
       - name: Upload test artifacts (on failure)
         uses: actions/upload-artifact@v3

diff --git a/.github/workflows/format.yaml b/.github/workflows/format.yaml
@@ -14,8 +14,8 @@ jobs:
     runs-on: ubuntu-20.04
     steps:
       - uses: actions/checkout@v3
-      - name: Install clang-format-10
-        run: sudo apt-get install clang-format-10
+      - name: Install clang-format-12
+        run: sudo apt-get install clang-format-12
       - uses: rhaschke/install-catkin_lint-action@v1.0
         with:
           distro: noetic

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -41,7 +41,7 @@ repos:
       - id: clang-format
         name: clang-format
         description: Format files with ClangFormat.
-        entry: clang-format-10
+        entry: clang-format-12
         language: system
         files: \.(c|cc|cxx|cpp|h|hpp|hxx|ih|ispc|ipp)$
         args: ["-fallback-style=none", "-i"]