mirror of
https://github.com/pchuan98/codex.git
synced 2026-07-01 00:31:56 +08:00
Adam Perry @ OpenAI
b17f30eb2a
## Why Made a mistake when carving #29746 out of my local changes and the test was missing from the build graph. Oops! ## What Enable the app-server Wine exec test target. Remove the `manual` tag from generated Wine-exec test variants so wildcard Bazel test invocations select them. Refactor the smoke test to ensure it passes with current Windows support.
37 lines
1.3 KiB
Bash
Executable File
37 lines
1.3 KiB
Bash
Executable File
#!/usr/bin/env bash
|
|
|
|
set -euo pipefail
|
|
|
|
repo_root="$(cd "$(dirname "${BASH_SOURCE[0]}")/../.." && pwd)"
|
|
cd "${repo_root}"
|
|
|
|
# Bazel wildcard builds skip manual targets, which misses the internal
|
|
# `*-unit-tests-bin` rust_test targets generated by `codex_rust_crate()`.
|
|
# Add only those manual rust_test targets explicitly so inline `#[cfg(test)]`
|
|
# call sites are linted without pulling in unrelated manual release targets.
|
|
manual_rust_test_targets="$(
|
|
./.github/scripts/run-bazel-query-ci.sh \
|
|
--output=label \
|
|
-- 'kind("rust_test rule", attr(tags, "manual", //codex-rs/...))'
|
|
)"
|
|
if [[ "${RUNNER_OS:-}" != "Windows" ]]; then
|
|
manual_rust_test_targets="$(printf '%s\n' "${manual_rust_test_targets}" | grep -v -- '-windows-cross-bin$' || true)"
|
|
fi
|
|
|
|
# Convert semantic lint opt-outs into negative target patterns so wildcard
|
|
# builds do not analyze toolchains used only by those wrappers.
|
|
excluded_targets="$(
|
|
./.github/scripts/run-bazel-query-ci.sh \
|
|
--output=label \
|
|
-- 'attr(tags, "no-argument-comment-lint", //codex-rs/...)'
|
|
)"
|
|
|
|
# The lint configuration does not register the transitioned Windows toolchain.
|
|
printf '%s\n' \
|
|
"//codex-rs/..." \
|
|
"-//codex-rs/core/tests/remote_env_windows:smoke-test"
|
|
if [[ -n "${excluded_targets}" ]]; then
|
|
printf '%s\n' "${excluded_targets}" | sed 's/^/-/'
|
|
fi
|
|
printf '%s\n' "${manual_rust_test_targets}"
|