Skip to content

Commit

Permalink
Clean up the Humble GitHub workflow. (#1884)
Browse files Browse the repository at this point in the history
* It should be testing humble, and should be using jammy.

* Also remove the cppcheck linter.

It is disabled in our CI anyway, because it is far
too slow nowadays.

Signed-off-by: Chris Lalancette <[email protected]>
  • Loading branch information
clalancette authored Dec 17, 2024
1 parent 33bb47c commit 95cf67d
Showing 1 changed file with 10 additions and 10 deletions.
20 changes: 10 additions & 10 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ jobs:
name: ament_${{ matrix.linter }}
runs-on: ubuntu-latest
container:
image: rostooling/setup-ros-docker:ubuntu-focal-ros-rolling-ros-base-latest
image: rostooling/setup-ros-docker:ubuntu-jammy-ros-humble-ros-base-latest
strategy:
fail-fast: false
matrix:
Expand All @@ -18,7 +18,7 @@ jobs:
- uses: ros-tooling/[email protected]
with:
linter: ${{ matrix.linter }}
distribution: rolling
distribution: humble
package-name: |
ros2bag
rosbag2
Expand All @@ -38,17 +38,17 @@ jobs:
name: ament_${{ matrix.linter }}
runs-on: ubuntu-latest
container:
image: rostooling/setup-ros-docker:ubuntu-focal-ros-rolling-ros-base-latest
image: rostooling/setup-ros-docker:ubuntu-jammy-ros-humble-ros-base-latest
strategy:
fail-fast: false
matrix:
linter: [cppcheck, cpplint, uncrustify]
linter: [cpplint, uncrustify]
steps:
- uses: actions/checkout@v4
- uses: ros-tooling/[email protected]
with:
linter: ${{ matrix.linter }}
distribution: rolling
distribution: humble
package-name: |
rosbag2_compression
rosbag2_compression_zstd
Expand All @@ -65,11 +65,11 @@ jobs:
name: ament_${{ matrix.linter }}
runs-on: ubuntu-latest
container:
image: rostooling/setup-ros-docker:ubuntu-focal-ros-rolling-ros-base-latest
image: rostooling/setup-ros-docker:ubuntu-jammy-ros-humble-ros-base-latest
strategy:
fail-fast: false
matrix:
linter: [cppcheck, cpplint, clang_format]
linter: [cpplint, clang_format]
include:
- linter: clang_format
arguments: "--config rosbag2_storage_mcap/.clang-format"
Expand All @@ -79,14 +79,14 @@ jobs:
with:
linter: ${{ matrix.linter }}
arguments: ${{ matrix.arguments }}
distribution: rolling
distribution: humble
package-name: rosbag2_storage_mcap

ament_lint_python: # Linters applicable to Python packages
name: ament_${{ matrix.linter }}
runs-on: ubuntu-latest
container:
image: rostooling/setup-ros-docker:ubuntu-focal-ros-rolling-ros-base-latest
image: rostooling/setup-ros-docker:ubuntu-jammy-ros-humble-ros-base-latest
strategy:
fail-fast: false
matrix:
Expand All @@ -96,7 +96,7 @@ jobs:
- uses: ros-tooling/[email protected]
with:
linter: ${{ matrix.linter }}
distribution: rolling
distribution: humble
package-name: |
ros2bag
rosbag2_py

0 comments on commit 95cf67d

Please sign in to comment.