openai/codex
1
0
Fork 0
mirror of https://github.com/openai/codex.git synced 2026-11-05 20:10:33 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: address coverage gap in argument-comment-lint

Browse Source
This commit is contained in:
Michael Bolin
2026-03-30 08:50:31 -07:00
parent 716f7b0428
commit 8423981666
2 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.bazelrc
+1
View File
@@ -80,6 +80,7 @@ build:clippy --@rules_rust//rust/settings:clippy.toml=//codex-rs:clippy.toml
# Shared config for Bazel-backed argument-comment-lint.
build:argument-comment-lint --aspects=//tools/argument-comment-lint:lint_aspect.bzl%rust_argument_comment_lint_aspect
build:argument-comment-lint --output_groups=argument_comment_lint_checks
build:argument-comment-lint --build_manual_tests
build:argument-comment-lint --@rules_rust//rust/toolchain/channel=nightly
# Rearrange caches on Windows so they're on the same volume as the checkout.
tools/argument-comment-lint/README.md
+3 -1
View File
@@ -138,7 +138,9 @@ If no package selection is provided, `just argument-comment-lint` now defaults
to the Bazel aspect path over `//codex-rs/...`. The Python wrappers remain the
package-scoped escape hatch and still default the underlying Cargo invocation
to `--all-targets` unless you explicitly narrow the target set, so targeted
wrapper runs cover test-only call sites by default.
wrapper runs cover test-only call sites by default. The Bazel config also sets
`--build_manual_tests` so wildcard lint runs include the internal
`*-unit-tests-bin` Rust targets and catch inline `#[cfg(test)]` call sites.
Repo runs also promote `uncommented_anonymous_literal_argument` to an error by
default: