Skip to content

Commit 0470c7c

Browse files
committed
drop! ci: run win+VS jobs on forks too
The `vs-build` job (and transitively the `vs-test` matrix, which declares `needs: [ci-config, vs-build]`) is gated on `github.event.repository.owner.login == 'git-for-windows'`, so a push to any other fork skips both. That means t7508 msysgit#126 ("status does not re-read unchanged 4 or 8 GiB file"), which only stresses the MSVC-only `ftruncate -> _chsize` compat shim, has no CI coverage when this topic is verified from a personal fork. Drop the owner check so the MSVC build runs here too. This commit is intentionally a no-op for `git-for-windows/git` itself; it only unblocks fork verification. Drop this commit before squashing the topic for upstream submission; upstream Git has no `vs-test` job and the gate is specific to the Git for Windows fork's CI matrix. Assisted-by: Opus 4.7 Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
1 parent dc5790c commit 0470c7c

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/workflows/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ jobs:
164164
vs-build:
165165
name: win+VS build
166166
needs: ci-config
167-
if: github.event.repository.owner.login == 'git-for-windows' && needs.ci-config.outputs.enabled == 'yes'
167+
if: needs.ci-config.outputs.enabled == 'yes'
168168
env:
169169
NO_PERL: 1
170170
GIT_CONFIG_PARAMETERS: "'user.name=CI' 'user.email=ci@git'"

0 commit comments

Comments
 (0)