Merge pull request #10 from esteve/fix-ci

Report errors on CI
tier4 · Nov 12, 2020 · 2f6bc7a · 2f6bc7a
2 parents d281b70 + ed547c0
commit 2f6bc7a
Showing 1 changed file with 15 additions and 4 deletions.
diff --git a/.github/workflows/build_and_test.yml b/.github/workflows/build_and_test.yml
@@ -2,24 +2,35 @@ name: Build and test
 
 on:
   pull_request:
+    branches:
+      - ros2
   push:
     branches:
       - ros2
 
 jobs:
   build-and-test:
-    runs-on: ubuntu-20.04
+    runs-on: ubuntu-latest
     container: ros:foxy
 
     steps:
     - name: Check out repo
       uses: actions/checkout@v2
 
     - name: Install missing dependencies
-      run: rosdep update && DEBIAN_FRONTEND=noninteractive sudo rosdep install --from-paths . --ignore-src --rosdistro foxy -y
+      run: |
+        sudo apt update
+        rosdep update
+        DEBIAN_FRONTEND=noninteractive rosdep install --from-paths . --ignore-src --rosdistro foxy -y
 
     - name: Build
-      run: . /opt/ros/foxy/setup.sh && colcon build --event-handlers console_cohesion+
+      run: |
+        . /opt/ros/foxy/setup.sh
+        colcon build --event-handlers console_cohesion+
 
     - name: Run tests
-      run: . /opt/ros/foxy/setup.sh && colcon test --event-handlers console_cohesion+
+      run: |
+        . /opt/ros/foxy/setup.sh
+        colcon test \
+          --event-handlers console_cohesion+ \
+          --return-code-on-test-failure