Skip to content

Commit 6dbff49

Browse files
committed
ci: make the self-hosted windows build manual-only
build-windows targets a self-hosted runner (Windows host + Linux docker images; GitHub-hosted Windows runners cannot run Linux containers). Drop the push trigger — it only queued-then-cancelled while the runner was offline — and the pull_request trigger, which would let fork PRs execute on the self-hosted machine. Hosted Windows coverage stays in test-core's windows lane.
1 parent dca094a commit 6dbff49

1 file changed

Lines changed: 5 additions & 17 deletions

File tree

.github/workflows/build-windows.yml

Lines changed: 5 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,11 @@
11
name: build windows
2+
# Runs on a SELF-HOSTED Windows runner (label: Windows) with Docker Desktop running
3+
# Linux images — GitHub-hosted Windows runners cannot run Linux containers, so this
4+
# is manual-only: no push trigger (queued-then-cancelled noise while the runner is
5+
# offline) and no pull_request trigger (public repo: fork PRs must never execute on
6+
# a self-hosted machine). Hosted Windows coverage lives in test-core's windows lane.
27
on:
38
workflow_dispatch:
4-
push:
5-
branches:
6-
- main
7-
paths:
8-
- cppjs-core/**
9-
- cppjs-extensions/**
10-
- cppjs-packages/**
11-
- cppjs-plugins/**
12-
- cppjs-samples/**
13-
- .github/workflows/build-windows.yml
14-
pull_request:
15-
branches:
16-
- main
17-
paths:
18-
- cppjs-core/cpp.js/**
19-
- cppjs-plugins/**
20-
- .github/workflows/build-windows.yml
219

2210
permissions: {}
2311
jobs:

0 commit comments

Comments
 (0)