Skip to content

Buffer stdout in ruff format - #29631

Merged
charliermarsh merged 2 commits into
mainfrom
charlie/buffer-ruff-format-output
Oct 22, 2026
Merged

charliermarsh merged 2 commits into
mainfrom
charlie/buffer-ruff-format-output

Conversation

@charliermarsh

@charliermarsh charliermarsh commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

Buffer stdout for ruff format --diff and ruff format --check.

On Linux, piping the diff for CPython’s Lib/ went from 4.3 s to 1.5 s.

@charliermarsh charliermarsh added performance Potential performance improvement formatter Related to the Ruff formatter labels Oct 22, 2026
@astral-automations-bot

Copy link
Copy Markdown
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.


Results for 8b454fa · Workflow run

@charliermarsh
charliermarsh marked this pull request as ready for review October 22, 2026 18:17
Comment thread crates/ruff/src/commands/format.rs Outdated
@charliermarsh charliermarsh changed the title Buffer redirected output from ruff format Buffer stdout in ruff format Oct 22, 2026
@charliermarsh
charliermarsh enabled auto-merge (squash) October 22, 2026 18:55
@charliermarsh
charliermarsh merged commit cc804d5 into main Oct 22, 2026
53 checks passed
@charliermarsh
charliermarsh deleted the charlie/buffer-ruff-format-output branch October 22, 2026 19:01

This branch was previously deployed

1 inactive deployment
automations — f8d90c7e Deployed Oct 22, 2026 by charliermarsh via security-review / security review #92927
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

formatter Related to the Ruff formatter performance Potential performance improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants