Skip to content

Commit f346a09

Browse files
committed
build: exclude namespace CLI test from affected-test runs
The `namespace` CLI test spawns child processes that serialize the entire stdlib namespace, routinely exceeding the 30-minute timeout in CI. Since `namespace` depends on nearly every stdlib package, it gets pulled into affected-test runs for almost any package change, causing unrelated timeouts. Assisted-by: Claude Opus 4.6 <noreply@anthropic.com> --- type: pre_commit_static_analysis_report description: Results of running static analysis checks when committing changes. report: - task: lint_filenames status: passed - task: lint_editorconfig status: passed - task: lint_markdown status: na - task: lint_package_json status: na - task: lint_repl_help status: na - task: lint_javascript_src status: na - task: lint_javascript_cli status: na - task: lint_javascript_examples status: na - task: lint_javascript_tests status: na - task: lint_javascript_benchmarks status: na - task: lint_python status: na - task: lint_r status: na - task: lint_c_src status: na - task: lint_c_examples status: na - task: lint_c_benchmarks status: na - task: lint_c_tests_fixtures status: na - task: lint_shell status: missing_dependencies - task: lint_typescript_declarations status: passed - task: lint_typescript_tests status: na - task: lint_license_headers status: passed ---
1 parent a0bec0e commit f346a09

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

  • .github/workflows/scripts/run_affected_tests

.github/workflows/scripts/run_affected_tests/run

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -164,6 +164,9 @@ main() {
164164
# Exclude files residing in test fixtures directories:
165165
files=$(echo "${files}" | grep -v '/fixtures/') || true
166166

167+
# Exclude the namespace CLI test, as it spawns child processes that dump the entire namespace and is too slow for affected-test runs:
168+
files=$(echo "${files}" | tr ' ' '\n' | grep -v '@stdlib/namespace/test/test.cli.js' | tr '\n' ' ') || true
169+
167170
if [[ -n "${files}" ]]; then
168171
# Invoke make in batches to avoid "Argument list too long" errors:
169172
printf '%s\n' ${files} | xargs sh -c 'make test-javascript-files-min FILES="$*"' _

0 commit comments

Comments
 (0)