Skip to content

Commit

Permalink
Merge pull request ros-perception#117 from ros-perception/feature/ci-…
Browse files Browse the repository at this point in the history
…kinetic-and-melodic

Run CI on kinetic and melodic as well as noetic
  • Loading branch information
jonbinney authored Jun 20, 2021
2 parents 353392e + 6df3d9d commit 7c77be2
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 3 deletions.
10 changes: 8 additions & 2 deletions .github/workflows/ros1_ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,15 +8,21 @@ on:

jobs:
build:
strategy:
matrix:
rosdistro: [kinetic, melodic, noetic]

runs-on: ubuntu-latest

continue-on-error: true

container:
# The perception docker images includes laser_geometry, which we need.
image: ros:noetic-perception
image: ros:${{ matrix.rosdistro }}-perception

steps:
- uses: actions/checkout@v2

- name: Build and run tests
run: . /opt/ros/noetic/setup.sh && ./ci.sh
run: . /opt/ros/${{ matrix.rosdistro }}/setup.sh && ./ci.sh

3 changes: 2 additions & 1 deletion ci.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,8 @@ set -e
# Should be run from the root directory of the repo.
BUILD_DIR=build

cmake -B ${BUILD_DIR} -DCATKIN_ENABLE_TESTING=1
mkdir -p ${BUILD_DIR}
(cd ${BUILD_DIR} && cmake .. -DCATKIN_ENABLE_TESTING=1)

# Build.
make -C ${BUILD_DIR}
Expand Down

0 comments on commit 7c77be2

Please sign in to comment.