Skip to content

Conversation

@justinchuby
Copy link
Contributor

@justinchuby justinchuby commented Jan 16, 2025

Use ruff as the code formatter in place of black and isort since it is much faster, and as projects like PyTorch and ONNX have adopted ruff format as well.

This PR include only auto-fixed changes in formatting.

@justinchuby justinchuby requested a review from snnn January 16, 2025 15:13
Comment on lines +13 to +19
from op_test_utils import (
TestDataFeeds, # noqa: F401
check_model_correctness,
check_op_type_count,
check_op_type_order, # noqa: F401
check_qtype_by_node_type,
)

Check notice

Code scanning / CodeQL

Unused import Note test

Import of 'TestDataFeeds' is not used.
Import of 'check_op_type_order' is not used.
Comment on lines +13 to +19
from op_test_utils import (
TestDataFeeds,
check_model_correctness,
check_op_nodes, # noqa: F401
check_op_type_count,
check_qtype_by_node_type,
)

Check notice

Code scanning / CodeQL

Unused import Note test

Import of 'check_op_nodes' is not used.
@snnn snnn merged commit c7c8757 into main Jan 16, 2025
98 checks passed
@snnn snnn deleted the justinchu/ruff-format-2 branch January 16, 2025 19:14
carzh pushed a commit that referenced this pull request Jan 16, 2025
Use ruff as the code formatter in place of black and isort since it is
much faster, and as projects like PyTorch and ONNX have adopted ruff
format as well.

This PR include only auto-fixed changes in formatting.
guschmue pushed a commit that referenced this pull request Mar 6, 2025
Use ruff as the code formatter in place of black and isort since it is
much faster, and as projects like PyTorch and ONNX have adopted ruff
format as well.

This PR include only auto-fixed changes in formatting.
ashrit-ms pushed a commit that referenced this pull request Mar 17, 2025
Use ruff as the code formatter in place of black and isort since it is
much faster, and as projects like PyTorch and ONNX have adopted ruff
format as well.

This PR include only auto-fixed changes in formatting.
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.

3 participants