Skip to content

Commit 02d6ac3

Browse files
perf: Only install yq on Windows if used (#362)
1 parent e938fdf commit 02d6ac3

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,8 +64,8 @@ steps:
6464
>
6565
> Using `flutter-version-file` requires [`yq`](https://github.com/mikefarah/yq),
6666
> which is not pre-installed in `windows` runners. Fortunately, since version
67-
> 2.18.0, this action installs `yq` automatically, so no action is required from
68-
> you.
67+
> 2.18.0, this action installs `yq` automatically if `flutter-version-file`
68+
> is specified, so no action is required from you.
6969

7070
### Use latest release for particular channel
7171

action.yaml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,11 +80,12 @@ outputs:
8080
runs:
8181
using: composite
8282
steps:
83-
# This is a cross-platform composite action that needs yq.
83+
# This is a cross-platform composite action that needs yq in order to parse
84+
# the pubspec.yaml file.
8485
# It's not preinstalled on Windows runners.
8586
# See https://github.com/actions/runner-images/issues/7443#issuecomment-1514597691
8687
- name: Make yq tool available on Windows runners
87-
if: runner.os == 'Windows'
88+
if: runner.os == 'Windows' && inputs.flutter-version-file != ''
8889
run: choco install yq
8990
shell: bash
9091

0 commit comments

Comments
 (0)