Repository navigation
Commit 4814775
committed
Comment all Ruff rule codes; tweak formatting
This adds comments to all specific Ruff rules suppressed in
pyproject.toml (most of which are global but one is for the tests
only). Some already had such comments but now all do.
This also harmonizes the toml formatting style with the rest of
pyproject.toml.1 parent 00ff7e3 commit 4814775
1 file changed
Lines changed: 17 additions & 13 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -54,24 +54,28 @@ exclude = [ | |||
| 54 | 54 | # Enable Pyflakes `E` and `F` codes by default. | |
| 55 | 55 | lint.select = [ | |
| 56 | 56 | "E", | |
| 57 | - "W", # see: https://pypi.org/project/pycodestyle | ||
| 58 | - "F", # see: https://pypi.org/project/pyflakes | ||
| 59 | - # "I", #see: https://pypi.org/project/isort/ | ||
| 60 | - # "S", # see: https://pypi.org/project/flake8-bandit | ||
| 61 | - # "UP", # see: https://docs.astral.sh/ruff/rules/#pyupgrade-up | ||
| 57 | + "W", # See: https://pypi.org/project/pycodestyle | ||
| 58 | + "F", # See: https://pypi.org/project/pyflakes | ||
| 59 | + # "I", # See: https://pypi.org/project/isort/ | ||
| 60 | + # "S", # See: https://pypi.org/project/flake8-bandit | ||
| 61 | + # "UP", # See: https://docs.astral.sh/ruff/rules/#pyupgrade-up | ||
| 62 | 62 | ] | |
| 63 | 63 | lint.extend-select = [ | |
| 64 | - #"A", # see: https://pypi.org/project/flake8-builtins | ||
| 65 | - "B", # see: https://pypi.org/project/flake8-bugbear | ||
| 66 | - "C4", # see: https://pypi.org/project/flake8-comprehensions | ||
| 67 | - "TCH004", # see: https://docs.astral.sh/ruff/rules/runtime-import-in-type-checking-block/ | ||
| 64 | + # "A", # See: https://pypi.org/project/flake8-builtins | ||
| 65 | + "B", # See: https://pypi.org/project/flake8-bugbear | ||
| 66 | + "C4", # See: https://pypi.org/project/flake8-comprehensions | ||
| 67 | + "TCH004", # See: https://docs.astral.sh/ruff/rules/runtime-import-in-type-checking-block/ | ||
| 68 | 68 | ] | |
| 69 | 69 | lint.ignore = [ | |
| 70 | - "E203", | ||
| 71 | - "E731", # Do not assign a `lambda` expression, use a `def` | ||
| 70 | + "E203", # Whitespace before ':' | ||
| 71 | + "E731", # Do not assign a `lambda` expression, use a `def` | ||
| 72 | 72 | ] | |
| 73 | 73 | lint.ignore-init-module-imports = true | |
| 74 | - lint.unfixable = ["F401"] | ||
| 74 | + lint.unfixable = [ | ||
| 75 | + "F401", # Module imported but unused | ||
| 76 | + ] | ||
| 75 | 77 | ||
| 76 | 78 | [tool.ruff.lint.per-file-ignores] | |
| 77 | - "test/**" = ["B018"] | ||
| 79 | + "test/**" = [ | ||
| 80 | + "B018", # useless-expression | ||
| 81 | + ] | ||
0 commit comments