Skip to content

Commit b3ab95f

Browse files
per1234per1234
authored
Merge pull request #123 from per1234/empty-sequence-syntax
Use correct syntax for empty YAML sequence
2 parents c9dd641 + 1c8eb83 commit b3ab95f

File tree

4 files changed

+33
-16
lines changed

4 files changed

+33
-16
lines changed

‎.github/workflows/test-integration.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ jobs:
7272
libraries: |
7373
# The official Servo library is not compatible with ESP8266, but that platform has a bundled Servo lib
7474
# so there are no library dependencies
75-
-
75+
[]
7676
7777
steps:
7878
- name: Checkout
@@ -348,7 +348,7 @@ jobs:
348348
fqbn: arduino:avr:uno
349349
github-token: ${{ secrets.GITHUB_TOKEN }}
350350
libraries: |
351-
-
351+
[]
352352
sketch-paths: |
353353
- ${{ env.TESTDATA_SKETCHES_PATH }}/Error
354354

‎README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
[![Check Action Metadata status](https://github.com/arduino/compile-sketches/actions/workflows/check-action-metadata-task.yml/badge.svg)](https://github.com/arduino/compile-sketches/actions/workflows/check-action-metadata-task.yml)
44
[![Check Files status](https://github.com/arduino/compile-sketches/actions/workflows/check-files-task.yml/badge.svg)](https://github.com/arduino/compile-sketches/actions/workflows/check-files-task.yml)
55
[![Check General Formatting status](https://github.com/arduino/compile-sketches/actions/workflows/check-general-formatting-task.yml/badge.svg)](https://github.com/arduino/compile-sketches/actions/workflows/check-general-formatting-task.yml)
6-
[![Check License status](https://github.com/arduino/compile-sketches/actions/workflows/check-license.yml/badge.svg)](https://github.com/arduino/compile-sketches/actions/workflows/check-license.ym
6+
[![Check License status](https://github.com/arduino/compile-sketches/actions/workflows/check-license.yml/badge.svg)](https://github.com/arduino/compile-sketches/actions/workflows/check-license.yml)
77
[![Check Markdown status](https://github.com/arduino/compile-sketches/actions/workflows/check-markdown-task.yml/badge.svg)](https://github.com/arduino/compile-sketches/actions/workflows/check-markdown-task.yml)
88
[![Check npm status](https://github.com/arduino/compile-sketches/actions/workflows/check-npm-task.yml/badge.svg)](https://github.com/arduino/compile-sketches/actions/workflows/check-npm-task.yml)
99
[![Check Prettier Formatting status](https://github.com/arduino/compile-sketches/actions/workflows/check-prettier-formatting-task.yml/badge.svg)](https://github.com/arduino/compile-sketches/actions/workflows/check-prettier-formatting-task.yml)
@@ -129,7 +129,7 @@ Keys:
129129
[YAML](https://en.wikipedia.org/wiki/YAML)-format list of library dependencies to install.
130130

131131
**Default**: `"- source-path: ./"`
132-
This causes the repository to be installed as a library. If there are no library dependencies and you want to override the default, set the `libraries` input to an empty list (`- libraries: '-'`).
132+
This causes the repository to be installed as a library. If there are no library dependencies and you want to override the default, set the `libraries` input to an empty list (`- libraries: '[]'`).
133133
134134
Libraries are installed under the Arduino user folder at `~/Arduino/libraries`.
135135

‎action.yml

Lines changed: 28 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,48 +1,64 @@
11
name: "Compile Arduino Sketches"
2-
description: "Checks whether Arduino sketches will compile and produces a report of data from the compilations"
2+
description: >-
3+
Checks whether Arduino sketches will compile and produces a report of data from the compilations
34
inputs:
45
cli-version:
5-
description: "Version of Arduino CLI to use when building"
6+
description: >-
7+
Version of Arduino CLI to use when building
68
default: "latest"
79
required: true
810
fqbn:
9-
description: "Full qualified board name, with Boards Manager URL if needed"
11+
description: >-
12+
Full qualified board name, with Boards Manager URL if needed
1013
default: "arduino:avr:uno"
1114
required: true
1215
libraries:
13-
description: "YAML-format list of library dependencies to install"
16+
description: >-
17+
YAML-format list of library dependencies to install
1418
default: "- source-path: ./"
1519
required: true
1620
platforms:
17-
description: "YAML-format list of platform dependencies to install"
21+
description: >-
22+
YAML-format list of platform dependencies to install
1823
default: ""
1924
required: true
2025
sketch-paths:
21-
description: "YAML-format list of paths containing sketches to compile."
26+
description: >-
27+
YAML-format list of paths containing sketches to compile.
2228
default: "- examples"
2329
required: true
2430
cli-compile-flags:
25-
description: "YAML-format list of flags to add to the Arduino CLI sketch compilation command."
31+
description: >-
32+
YAML-format list of flags to add to the Arduino CLI sketch compilation command.
2633
default: ""
2734
required: false
2835
verbose:
29-
description: "Set to true to show verbose output in the log"
36+
description: >-
37+
Set to true to show verbose output in the log
3038
default: "false"
3139
required: true
3240
sketches-report-path:
33-
description: "Path in which to save a JSON formatted file containing data from the sketch compilations"
41+
description: >-
42+
Path in which to save a JSON formatted file containing data from the sketch compilations
3443
default: "sketches-reports"
3544
required: true
3645
github-token:
37-
description: "GitHub access token used to get information from the GitHub API. Only needed if you are using the deltas report feature in a private repository."
46+
description: >-
47+
GitHub access token used to get information from the GitHub API.
48+
49+
Only needed if you are using the deltas report feature in a private repository.
3850
default: ""
3951
required: true
4052
enable-deltas-report:
41-
description: "Set to true to cause the action to determine the change in memory usage and compiler warnings of the compiled sketches between the head and base refs of a PR and the immediate parent commit of a push"
53+
description: >-
54+
Set to true to cause the action to determine the change in memory usage and compiler warnings of the compiled
55+
sketches between the head and base refs of a PR and the immediate parent commit of a push
4256
default: "false"
4357
required: true
4458
enable-warnings-report:
45-
description: "Set to true to cause the action to record the compiler warning count for each sketch compilation in the sketches report"
59+
description: >-
60+
Set to true to cause the action to record the compiler warning count for each sketch compilation in the sketches
61+
report
4662
default: "false"
4763
required: true
4864

‎compilesketches/tests/test_compilesketches.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1070,6 +1070,7 @@ def __init__(self):
10701070
[],
10711071
),
10721072
("-", [], [], [], []),
1073+
("[]", [], [], [], []),
10731074
(
10741075
"- " + compilesketches.CompileSketches.dependency_name_key + ": foo",
10751076
[{compilesketches.CompileSketches.dependency_name_key: "foo"}],

0 commit comments

Comments
 (0)