Skip to content

style(ci): format runner utilization report - #3590

Merged
guapisolo merged 1 commit into
radixark:mainfrom
michaelzhang-ai:fix/format-runner-utilization-report
Sep 22, 2026
Merged

guapisolo merged 1 commit into
radixark:mainfrom
michaelzhang-ai:fix/format-runner-utilization-report

Conversation

@michaelzhang-ai

@michaelzhang-ai michaelzhang-ai commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

What

Format scripts/ci/runner_utilization_report.py with the repository's pinned Black configuration. This is a formatting-only change; the parsed Python AST is unchanged.

Why

#3586 added the report at Black's default 88-column layout, while Miles configures 119 columns. Its pre-commit check failed on the file, but the PR was merged, so the first main-branch run failed as well.

The workflow checks all files, so unrelated branches inherit the failure. For example, #3588's pre-commit job rewrites this file even though that PR only changes the ROCm Dockerfile.

Test plan

  • pre-commit run black --files scripts/ci/runner_utilization_report.py
  • python -m py_compile scripts/ci/runner_utilization_report.py
  • Verified the parsed Python AST is identical before and after formatting
  • Full GitHub Actions pre-commit job
  • PR Test CPU workflow (all five CPU jobs passed)

PR radixark#3586 added the report at Black's default 88-column layout, while
Miles configures 119 columns. Because pre-commit checks all files, the
mismatch leaves main and unrelated pull requests red.

Apply the repository's pinned Black 24.3.0 formatter. The parsed Python
AST is unchanged.

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
@michaelzhang-ai
michaelzhang-ai marked this pull request as ready for review September 22, 2026 03:38

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@michaelzhang-ai

Copy link
Copy Markdown
Contributor Author

@yushengsu-thu

@guapisolo
guapisolo merged commit 418be99 into radixark:main Sep 22, 2026
40 checks passed
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