Skip to content

Conversation

@junrushao
Copy link
Member

@junrushao junrushao commented Sep 17, 2025

Enables a subset of Ruff rules including:

select = [
  "UP",  # pyupgrade, https://docs.astral.sh/ruff/rules/#pyupgrade-up
  "PL",  # pylint, https://docs.astral.sh/ruff/rules/#pylint-pl
  "I",   # isort, https://docs.astral.sh/ruff/rules/#isort-i
  "RUF", # ruff, https://docs.astral.sh/ruff/rules/#ruff-specific-rules-ruf
  "NPY", # numpy, https://docs.astral.sh/ruff/rules/#numpy-specific-rules-npy
  "F",   # pyflakes, https://docs.astral.sh/ruff/rules/#pyflakes-f
  # "ANN", # flake8-annotations, https://docs.astral.sh/ruff/rules/#flake8-annotations-ann
  "PTH", # flake8-use-pathlib, https://docs.astral.sh/ruff/rules/#flake8-use-pathlib-pth
  "D", # pydocstyle, https://docs.astral.sh/ruff/rules/#pydocstyle-d
]

@junrushao junrushao force-pushed the 2025-09-17/pre-commit-hooks-2 branch from 17efe6e to 73b20a1 Compare September 17, 2025 21:34
@junrushao junrushao force-pushed the 2025-09-17/pre-commit-hooks-2 branch from e9793d0 to 63cc7cf Compare September 17, 2025 22:01
@junrushao junrushao marked this pull request as ready for review September 17, 2025 22:05
@junrushao junrushao requested a review from tqchen September 17, 2025 22:05
@junrushao junrushao force-pushed the 2025-09-17/pre-commit-hooks-2 branch from 5fecd56 to 81317cb Compare September 17, 2025 22:19
@junrushao junrushao changed the title chore: more modernization chore: Basic Ruff rule coverage Sep 17, 2025
@junrushao junrushao force-pushed the 2025-09-17/pre-commit-hooks-2 branch from 81317cb to 906ded1 Compare September 17, 2025 22:26
@tqchen tqchen merged commit 0bc968d into apache:main Sep 17, 2025
6 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