Skip to content

Commit ea0f2a8

Browse files
ai: apply changes for #879 (1 review thread)
Addresses: - #3641857632 at .github/workflows/trigger-integration-tests.yml:479 Signed-off-by: peco-engineer-bot[bot] <peco-engineer-bot[bot]@users.noreply.github.com>
1 parent f3d4d41 commit ea0f2a8

1 file changed

Lines changed: 8 additions & 1 deletion

File tree

.github/workflows/trigger-integration-tests.yml

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -485,7 +485,14 @@ jobs:
485485
// The whole repo IS the driver. Count a merge as source-affecting when it changes a file under src/.
486486
// Docs/CI/test-only merges do not warrant a full multi-language fan-out.
487487
const isSource = (f) => f.startsWith('src/');
488-
const srcChanged = files.some((f) => isSource(f.filename));
488+
// GitHub caps pulls.listFiles at 3000 files per PR (even via paginate). If a merge is that
489+
// large the list is truncated, so a src/ file could sort beyond the cap and be missed. Since
490+
// the whole gate hinges on this boolean, treat a truncated result set as source-affecting.
491+
const truncated = files.length >= 3000;
492+
const srcChanged = truncated || files.some((f) => isSource(f.filename));
493+
if (truncated) {
494+
console.log('listFiles hit the 3000-file cap; treating merge as source-affecting.');
495+
}
489496
console.log(`driver source changed: ${srcChanged}`);
490497
core.setOutput('source', srcChanged.toString());
491498

0 commit comments

Comments
 (0)