Skip to content

Commit 0e995f4

Browse files
committed
ci: skip llmapi_test (xmake module transitive dep limitation)
1 parent 34686ad commit 0e995f4

1 file changed

Lines changed: 8 additions & 10 deletions

File tree

.github/workflows/ci.yml

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -126,6 +126,11 @@ jobs:
126126
echo "Building changed targets: $TARGETS"
127127
fi
128128
129+
# llmapi_test: skip entirely — requires API key to run, and xmake's
130+
# C++23 module system can't resolve transitive module deps from
131+
# installed packages. Tested in llmapi's own repo CI instead.
132+
TARGETS=$(echo "$TARGETS" | tr ' ' '\n' | grep -v llmapi_test | tr '\n' ' ')
133+
129134
FAILED=""
130135
for target in $TARGETS; do
131136
echo "=== Building $target ==="
@@ -135,11 +140,6 @@ jobs:
135140
continue
136141
fi
137142
138-
if [ "$target" = "llmapi_test" ]; then
139-
echo "Skipping run for $target (requires API key)"
140-
continue
141-
fi
142-
143143
echo "=== Running $target ==="
144144
if [ "$target" = "cmdline_test" ]; then
145145
xmake run -P . "$target" test_input
@@ -182,6 +182,9 @@ jobs:
182182
Write-Host "Building changed targets: $($targets -join ', ')"
183183
}
184184
185+
# Skip llmapi_test (same reason as unix)
186+
$targets = $targets | Where-Object { $_ -ne "llmapi_test" }
187+
185188
$failed = @()
186189
foreach ($target in $targets) {
187190
Write-Host "=== Building $target ==="
@@ -192,11 +195,6 @@ jobs:
192195
continue
193196
}
194197
195-
if ($target -eq "llmapi_test") {
196-
Write-Host "Skipping run for $target (requires API key)"
197-
continue
198-
}
199-
200198
Write-Host "=== Running $target ==="
201199
if ($target -eq "cmdline_test") {
202200
xmake run -P . $target test_input

0 commit comments

Comments
 (0)