Skip to content

Commit 0c1de5b

Browse files
Fix weekly CI (backport #58) (#62)
(cherry picked from commit d7b5088) Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com> Co-authored-by: David Laseca Perez <davidlaseca@eprosima.com>
1 parent ca42dde commit 0c1de5b

6 files changed

Lines changed: 7 additions & 14 deletions

File tree

.github/workflows/humble-ci.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ jobs:
1010
humble-ci:
1111
uses: ./.github/workflows/reusable-ci.yml
1212
with:
13-
branch: ${{ github.ref }}
1413
os: ubuntu-22.04
1514
docker-image: ubuntu:jammy
1615
ros-distribution: humble

.github/workflows/jazzy-ci.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ jobs:
1010
jazzy-ci:
1111
uses: ./.github/workflows/reusable-ci.yml
1212
with:
13-
branch: ${{ github.ref }}
1413
os: ubuntu-24.04
1514
docker-image: ubuntu:noble
1615
ros-distribution: jazzy

.github/workflows/kilted-ci.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ jobs:
1010
kilted-ci:
1111
uses: ./.github/workflows/reusable-ci.yml
1212
with:
13-
branch: ${{ github.ref }}
1413
os: ubuntu-24.04
1514
docker-image: ubuntu:noble
1615
ros-distribution: kilted

.github/workflows/lyrical-ci.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ jobs:
1010
lyrical-ci:
1111
uses: ./.github/workflows/reusable-ci.yml
1212
with:
13-
branch: ${{ github.ref }}
1413
os: ubuntu-26.04
1514
docker-image: ubuntu:resolute
1615
ros-distribution: lyrical

.github/workflows/reusable-ci.yml

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,9 @@ on:
44
workflow_call:
55
inputs:
66
branch:
7-
description: "The rcutils branch to use for the workflow"
8-
required: true
7+
description: "The rcutils branch to build. Leave unset for push and pull_request."
8+
required: false
9+
default: ""
910
type: string
1011
os:
1112
description: "The OS to use for the workflow"
@@ -33,13 +34,7 @@ jobs:
3334
apt-get update && apt-get install -y git
3435
shell: bash
3536
36-
- name: Sync repository
37-
uses: actions/checkout@v5
38-
with:
39-
ref: ${{ inputs.branch }}
40-
submodules: recursive
41-
42-
- name: Setup ROS 2
37+
- name: Setup ROS 2
4338
uses: ros-tooling/setup-ros@0.7.19
4439
with:
4540
required-ros-distributions: ${{ inputs.ros-distribution }}
@@ -49,8 +44,11 @@ jobs:
4944
apt-get install ros-${{ inputs.ros-distribution }}-mimick-vendor
5045
apt-get -y install ros-${{ inputs.ros-distribution }}-performance-test-fixture
5146
47+
# This action clones the repository itself, so no checkout step is needed.
48+
# When ref is empty it uses the branch that triggered the run.
5249
- uses : ros-tooling/action-ros-ci@0.4.8
5350
with:
5451
package-name: "rcutils"
5552
target-ros2-distro: ${{ inputs.ros-distribution }}
53+
ref: ${{ inputs.branch }}
5654
skip-tests: true

.github/workflows/rolling-ci.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ jobs:
1010
rolling-ci:
1111
uses: ./.github/workflows/reusable-ci.yml
1212
with:
13-
branch: ${{ github.ref }}
1413
os: ubuntu-24.04
1514
docker-image: ubuntu:noble
1615
ros-distribution: rolling

0 commit comments

Comments
 (0)