Skip to content

Convert directory fbcode/torchx to use the Ruff Formatter - #976

Closed
tpolasek wants to merge 1 commit into
meta-pytorch:mainfrom
tpolasek:export-D66248087
Closed

tpolasek wants to merge 1 commit into
meta-pytorch:mainfrom
tpolasek:export-D66248087

Conversation

@tpolasek

Copy link
Copy Markdown
Contributor

Summary:
Converts the directory specified to use the Ruff formatter in pyfmt

ruff_dog

If this diff causes merge conflicts when rebasing, please run
hg status -n -0 --change . -I '**/*.{py,pyi}' | xargs -0 arc pyfmt
on your diff, and amend any changes before rebasing onto latest.
That should help reduce or eliminate any merge conflicts.

allow-large-files

Reviewed By: amyreese

Differential Revision: D66248087

Summary:
Converts the directory specified to use the Ruff formatter in pyfmt

ruff_dog

If this diff causes merge conflicts when rebasing, please run
`hg status -n -0 --change . -I '**/*.{py,pyi}' | xargs -0 arc pyfmt`
on your diff, and amend any changes before rebasing onto latest.
That should help reduce or eliminate any merge conflicts.

allow-large-files

Reviewed By: amyreese

Differential Revision: D66248087
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 25, 2024
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D66248087

@kiukchung
kiukchung requested review from d4l3k and yikaiMeta and removed request for yikaiMeta December 2, 2024 21:01
@kiukchung

Copy link
Copy Markdown
Contributor

This landed via D66248087 (Nov 2024); closing this stale sync mirror.

@kiukchung kiukchung closed this Jul 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants