Skip to content

Commit 71aa464

Browse files
committed
Update action versions used in CI workflows
The "smoke test" sketch compilation and reporting actions have been moved to dedicated repositories. The actions hosted at the old `arduino/actions/libraries/*` location will no longer be maintained. Since the time the actions were migrated to the dedicated repositories, a breaking change was made to the default value of the `sketches-report-path` input, which required a change to the values of the `name` and `path` inputs of the `actions/upload-artifact` action.
1 parent aa6dcf7 commit 71aa464

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

.github/workflows/compile-examples.yml

+5-4
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ jobs:
2525
env:
2626
# sketch paths to compile (recursive) compatible with all boards
2727
UNIVERSAL_SKETCH_PATHS: '"libraries/Scheduler"'
28+
SKETCHES_REPORTS_PATH: sketches-reports
2829

2930
strategy:
3031
fail-fast: false
@@ -62,7 +63,7 @@ jobs:
6263
run: mv "$GITHUB_WORKSPACE/ArduinoCore-API/api" "$GITHUB_WORKSPACE/cores/arduino"
6364

6465
- name: Compile examples
65-
uses: arduino/actions/libraries/compile-examples@master
66+
uses: arduino/compile-sketches@main
6667
with:
6768
github-token: ${{ secrets.GITHUB_TOKEN }}
6869
fqbn: ${{ matrix.board.fqbn }}
@@ -78,7 +79,7 @@ jobs:
7879

7980
- name: Save memory usage change report as artifact
8081
if: github.event_name == 'pull_request'
81-
uses: actions/upload-artifact@v1
82+
uses: actions/upload-artifact@v2
8283
with:
83-
name: size-deltas-reports
84-
path: size-deltas-reports
84+
path: ${{ env.SKETCHES_REPORTS_PATH }}
85+
name: ${{ env.SKETCHES_REPORTS_PATH }}

.github/workflows/report-size-deltas.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,4 +8,4 @@ jobs:
88

99
steps:
1010
- name: Comment size deltas reports to PRs
11-
uses: arduino/actions/libraries/report-size-deltas@master
11+
uses: arduino/report-size-deltas@main

0 commit comments

Comments
 (0)