Skip to content

Test repository-wide Rust formatter discovery - #41925

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/0cfffde1bb91bf620d2118bed1efc68466c0cca6
Aug 31, 2026
Merged

Test repository-wide Rust formatter discovery#41925
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/0cfffde1bb91bf620d2118bed1efc68466c0cca6

Conversation

@copyberry

@copyberry copyberry Bot commented Aug 31, 2026

Copy link
Copy Markdown

Test repository-wide Rust formatter discovery

What changed

  • Exercise Rust source discovery through a mocked git ls-files call, including files outside codex-rs, paths containing spaces, and stale entries that no longer exist.
  • Assert that the formatter invokes rustfmt from codex-rs with the repository configuration in both format and check modes.
  • Keep the multiline argument-comment lint fixture stable under formatting and update affected Rust test files to the expected style.

## What changed

- Exercise Rust source discovery through a mocked `git ls-files` call, including files outside `codex-rs`, paths containing spaces, and stale entries that no longer exist.
- Assert that the formatter invokes `rustfmt` from `codex-rs` with the repository configuration in both format and check modes.
- Keep the multiline argument-comment lint fixture stable under formatting and update affected Rust test files to the expected style.

GitOrigin-RevId: 0cfffde1bb91bf620d2118bed1efc68466c0cca6
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/0cfffde1bb91bf620d2118bed1efc68466c0cca6 branch from bd25e16 to 9127f21 Compare August 31, 2026 22:49
@copyberry
copyberry Bot merged commit 9127f21 into main Aug 31, 2026
22 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/0cfffde1bb91bf620d2118bed1efc68466c0cca6 branch August 31, 2026 22:50
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 31, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants