Skip to content

Commit

Permalink
Add target_workspace to ICI workflow (#3)
Browse files Browse the repository at this point in the history
* Add target_workspace to ICI workflow

This should allow to optionally change the TARGET_WORKSPACE variable for ICI.

* Make upstream workspace optional

Since we want to make this file more usable, there's no need to make the flag required, while it is optional in ICI.

Co-authored-by: Christoph Fröhlich <[email protected]>

---------

Co-authored-by: Christoph Fröhlich <[email protected]>
  • Loading branch information
fmauch and christophfroehlich authored Feb 16, 2024
1 parent 2f069f3 commit a82f429
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion .github/workflows/reusable-industrial-ci-with-cache.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,12 @@ on:

upstream_workspace:
description: 'UPSTREAM_WORKSPACE variable for industrial_ci. Usually path to local .repos file.'
required: true
required: false
type: string
target_workspace:
description: 'TARGET_WORKSPACE variable for industrial_ci. If not provided, defaults to the current repo.'
required: false
default: ""
type: string
downstream_workspace:
description: 'DOWNSTREAM_WORKSPACE variable for industrial_ci. If set, downstream packages will be built and tested against this repo'
Expand Down Expand Up @@ -88,6 +93,7 @@ jobs:
- uses: 'ros-industrial/industrial_ci@master'
env:
UPSTREAM_WORKSPACE: ${{ inputs.upstream_workspace }}
TARGET_WORKSPACE: ${{ inputs.target_workspace }}
DOWNSTREAM_WORKSPACE: ${{ inputs.downstream_workspace }}
ROS_DISTRO: ${{ inputs.ros_distro }}
ROS_REPO: ${{ inputs.ros_repo }}
Expand Down

0 comments on commit a82f429

Please sign in to comment.