Skip to content

Check root Python script formatting in CI#25165

Open
anp-oai wants to merge 3 commits into
mainfrom
codex/format-python-scripts-ci
Open

Check root Python script formatting in CI#25165
anp-oai wants to merge 3 commits into
mainfrom
codex/format-python-scripts-ci

Conversation

@anp-oai
Copy link
Copy Markdown
Collaborator

@anp-oai anp-oai commented May 29, 2026

Why

Python files under scripts/ were not covered by the repository formatting recipe or the CI formatting job, so formatting drift could merge unnoticed.

What

  • Add a dedicated scripts/pyproject.toml and scripts/uv.lock so root-script formatting uses a locked Ruff version.
  • Extend just fmt to format root Python scripts and add fmt-scripts-check for CI.
  • Run just fmt-scripts-check from .github/workflows/ci.yml, installing uv through SHA-pinned astral-sh/setup-uv while retaining the uv 0.11.3 pin.
  • Apply Ruff formatting to the existing Python scripts and extend sdk/python/tests/test_artifact_workflow_and_binaries.py to cover the root formatting recipe.
  • Update AGENTS.md so agents run just fmt after code changes anywhere in the repository.

Validation

  • Extended the existing Python SDK workflow test to assert that just fmt includes root Python scripts.

@anp-oai anp-oai changed the title Check formatting for Python scripts Check root Python script formatting in CI May 30, 2026
Comment thread justfile Outdated
Comment thread justfile
@anp-oai
Copy link
Copy Markdown
Collaborator Author

anp-oai commented May 30, 2026

Waiting for #24983 before landing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants