Skip to content

Format Python scripts across the repository - #42109

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/73ac82b8bb9c713d589724f6c4f926399febb513
Sep 1, 2026
Merged

Format Python scripts across the repository#42109
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/73ac82b8bb9c713d589724f6c4f926399febb513

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 1, 2026

Copy link
Copy Markdown

Format Python scripts across the repository

Why

The root Python formatting pass only covered scripts/, leaving Python utilities elsewhere in the repository outside just fmt and just fmt-check.

What changed

  • Run the scripts Ruff formatter from the repository root.
  • Exclude sdk/python, which has its own formatter pass, and codex-rs/vendor.
  • Apply Ruff formatting to the newly covered Python files and update the formatter coverage test.

## Why

The root Python formatting pass only covered `scripts/`, leaving Python utilities elsewhere in the repository outside `just fmt` and `just fmt-check`.

## What changed

- Run the scripts Ruff formatter from the repository root.
- Exclude `sdk/python`, which has its own formatter pass, and `codex-rs/vendor`.
- Apply Ruff formatting to the newly covered Python files and update the formatter coverage test.

GitOrigin-RevId: 73ac82b8bb9c713d589724f6c4f926399febb513
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/73ac82b8bb9c713d589724f6c4f926399febb513 branch from 75866da to 84aa752 Compare September 1, 2026 17:48
@copyberry
copyberry Bot merged commit 84aa752 into main Sep 1, 2026
24 of 30 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/73ac82b8bb9c713d589724f6c4f926399febb513 branch September 1, 2026 17:49
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 1, 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.

1 participant