Skip to content

Commit

Permalink
make cache direction explicit, remove cache from colcon-test
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 27, 2024
1 parent 39b37a4 commit bcbc34a
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 19 deletions.
2 changes: 1 addition & 1 deletion clang-tidy/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ runs:

- name: Restore build files from cache
id: restore-build-files
uses: actions/cache@v4
uses: actions/cache/restore@v4
with:
path: |
./build
Expand Down
2 changes: 1 addition & 1 deletion colcon-build/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ runs:
shell: bash

- name: Cache build artifacts
uses: actions/cache@v4
uses: actions/cache/save@v4
with:
path: |
./build
Expand Down
17 changes: 0 additions & 17 deletions colcon-test/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,10 +11,6 @@ inputs:
build-depends-repos:
description: ""
required: false
cmake-build-type:
description: ""
required: false
default: Release
label-regex:
description: ""
required: false
Expand All @@ -26,10 +22,6 @@ 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 @@ -85,15 +77,6 @@ runs:
DEBIAN_FRONTEND=noninteractive rosdep install -yqq --from-paths ${package_paths} --ignore-src --rosdistro ${{ inputs.rosdistro }}
shell: bash

- name: Restore build files from cache
id: restore-build-files
uses: actions/cache@v4
with:
path: |
./build
./install
key: build-${{ runner.arch }}-${{ inputs.rosdistro }}-${{ inputs.cache-key-element }}-${{ inputs.cmake-build-type }}-${{ github.sha }}

- name: Test
run: |
set -e
Expand Down

0 comments on commit bcbc34a

Please sign in to comment.