Skip to content

Commit

Permalink
refactor: add missing input to colcon-test and add cache-key-element
Browse files Browse the repository at this point in the history
Signed-off-by: M. Fatih Cırıt <[email protected]>
  • Loading branch information
M. Fatih Cırıt committed Jun 26, 2024
1 parent 92b8188 commit 77d7655
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 3 deletions.
6 changes: 5 additions & 1 deletion clang-tidy/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,10 @@ inputs:
include-eol-distros:
description: ""
required: false
cache-key-element:
description: "Will be part of the cache key"
default: "default"
required: false

runs:
using: composite
Expand Down Expand Up @@ -88,7 +92,7 @@ runs:
path: |
./build
./install
key: build-${{ inputs.rosdistro }}-${{ runner.os }}-${{ runner.arch }}-${{ inputs.cmake-build-type }}-${{ github.sha }}
key: build-${{ inputs.rosdistro }}-${{ runner.arch }}-${{ inputs.cache-key-element }}-${{ inputs.cmake-build-type }}-${{ github.sha }}

- name: Build
if: ${{ steps.restore-build-files.outputs.cache-hit != 'true' }}
Expand Down
6 changes: 5 additions & 1 deletion colcon-build/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,10 @@ inputs:
include-eol-distros:
description: ""
required: false
cache-key-element:
description: "Will be part of the cache key"
default: "default"
required: false

runs:
using: composite
Expand Down Expand Up @@ -90,4 +94,4 @@ runs:
path: |
./build
./install
key: build-${{ inputs.rosdistro }}-${{ runner.os }}-${{ runner.arch }}-${{ inputs.cmake-build-type }}-${{ github.sha }}
key: build-${{ inputs.rosdistro }}-${{ runner.arch }}-${{ inputs.cache-key-element }}-${{ inputs.cmake-build-type }}-${{ github.sha }}
10 changes: 9 additions & 1 deletion colcon-test/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,10 @@ inputs:
build-depends-repos:
description: ""
required: false
cmake-build-type:
description: ""
required: false
default: Release
label-regex:
description: ""
required: false
Expand All @@ -22,6 +26,10 @@ inputs:
include-eol-distros:
description: ""
required: false
cache-key-element:
description: "Will be part of the cache key"
default: "default"
required: false
outputs:
coverage-report-files:
description: ""
Expand Down Expand Up @@ -84,7 +92,7 @@ runs:
path: |
./build
./install
key: build-${{ inputs.rosdistro }}-${{ runner.os }}-${{ runner.arch }}-${{ inputs.cmake-build-type }}-${{ github.sha }}
key: build-${{ inputs.rosdistro }}-${{ runner.arch }}-${{ inputs.cache-key-element }}-${{ inputs.cmake-build-type }}-${{ github.sha }}

- name: Test
run: |
Expand Down

0 comments on commit 77d7655

Please sign in to comment.