Skip to content

ci: Fix matrix to use testgui flag properly - #1570

Merged
ychin merged 1 commit into
macvim-dev:masterfrom
ychin:fix-ci-respect-matrix-testgui
Mar 28, 2025
Merged

ci: Fix matrix to use testgui flag properly#1570
ychin merged 1 commit into
macvim-dev:masterfrom
ychin:fix-ci-respect-matrix-testgui

Conversation

@ychin

Copy link
Copy Markdown
Member

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.

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 macvim-dev#1559, there was a mistake
where this matrix flag wasn't ported over properly to an input, causing
it to always evaluate to false.
@ychinychin added Infrastructure Non-app infrastructure issues, e.g. CI Non User Facing Non-user facing change. These issues do no need to show up in release notes. labels Mar 28, 2025
@ychinychin added this to the Release 182 milestone Mar 28, 2025
@github-actionsgithub-actionsBot added the CI Vim upstream label for CI issues label Mar 28, 2025
@ychin
ychin merged commit b2c10a6 into macvim-dev:masterMar 28, 2025
@ychin
ychin deleted the fix-ci-respect-matrix-testgui branch March 28, 2025 02:49
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CIVim upstream label for CI issuesInfrastructureNon-app infrastructure issues, e.g. CINon User FacingNon-user facing change. These issues do no need to show up in release notes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@ychin