Skip to content

Commit 3d76744

Browse files
committed
ci: Fix matrix to use testgui flag properly
This flag is there to get CI to only run either GUI or CLI tests, unless we are publishing a release, in order to cut down on interation time. When the YAML file was split into two in #1559, there was a mistake where this matrix flag wasn't ported over properly to an input, causing it to always evaluate to false.
1 parent 6b809a4 commit 3d76744

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

.github/workflows/ci-macvim.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ jobs:
5252
os: ${{ matrix.os }}
5353
legacy: ${{ matrix.legacy && true || false }}
5454
xcode: ${{ matrix.xcode }}
55+
testgui: ${{ matrix.testgui && true || false }}
5556
publish: ${{ matrix.publish && true || false }}
5657
publish_postfix: ${{ matrix.publish_postfix }}
5758
optimized: ${{ matrix.optimized && true || false }}

.github/workflows/macvim-buildtest.yaml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@ on:
1313
type: boolean
1414
xcode:
1515
type: string
16+
testgui:
17+
type: boolean
1618
publish:
1719
type: boolean
1820
publish_postfix:
@@ -363,7 +365,7 @@ jobs:
363365
make ${MAKE_BUILD_ARGS} -j${NPROC} -C src unittesttargets
364366
365367
- name: Test Vim
366-
if: startsWith(github.ref, 'refs/tags/') || !matrix.testgui
368+
if: startsWith(github.ref, 'refs/tags/') || !inputs.testgui
367369
timeout-minutes: 30
368370
run: |
369371
defaults delete org.vim.MacVim # Clean up stale states
@@ -373,7 +375,7 @@ jobs:
373375
make ${MAKE_BUILD_ARGS} -C src test
374376
375377
- name: Test Vim (GUI)
376-
if: startsWith(github.ref, 'refs/tags/') || matrix.testgui
378+
if: startsWith(github.ref, 'refs/tags/') || inputs.testgui
377379
timeout-minutes: 30
378380
run: |
379381
defaults delete org.vim.MacVim # Clean up stale states

0 commit comments

Comments
 (0)