Skip to content

style: format test_cumsum_aten.py with black - #4582

Open
shoumikhin wants to merge 1 commit into
pytorch:mainfrom
shoumikhin:fix/black-format-cumsum-test
Open

style: format test_cumsum_aten.py with black#4582
shoumikhin wants to merge 1 commit into
pytorch:mainfrom
shoumikhin:fix/black-format-cumsum-test

Conversation

@shoumikhin

Copy link
Copy Markdown
Contributor

What is broken

The Python Linting job runs black --check . over the whole repository. One
file is not formatted, so the job fails, and it fails the same way on every open
pull request no matter what that pull request changes:

would reformat tests/py/dynamo/conversion/test_cumsum_aten.py
1 file would be reformatted, 602 files would be left unchanged.

Fix

Run black on that file. Two call sites are split across several lines but fit on
one, so black joins them. Nothing else changes, and there is no behavior change.

Tested

black --check . no longer reports this file.

The Python Linting job runs `black --check .` over the whole repository and
fails the build when any file is not formatted. Two call sites in this test
file are split across several lines but fit on one, so black wants to rewrite
them, and the job has been red on every pull request since that file was last
changed.

Reformat the file. No behavior change.
@meta-cla meta-cla Bot added the cla signed label Aug 26, 2026
@github-actions github-actions Bot added the component: tests Issues re: Tests label Aug 26, 2026
@github-actions
github-actions Bot requested a review from lanluo-nvidia August 26, 2026 01:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant