Lint Python code with ruff

Author: cclauss

docs/contributing/2.-coding-standard.md

@@ -52,6 +52,7 @@ All code submitted to hug should run through the following tools:
 - Flake8
    - flake8-bugbear
 - Bandit
+- ruff
 - pep8-naming
 - vulture
 - safety

isort/settings.py

@@ -892,11 +892,11 @@ def _get_config_data(file_path: str, sections: Tuple[str, ...]) -> Dict[str, Any
 
         for key, value in settings.items():
             existing_value_type = _get_str_to_type_converter(key)
-            if existing_value_type == tuple:
+            if existing_value_type is tuple:
                 settings[key] = tuple(_as_list(value))
-            elif existing_value_type == frozenset:
+            elif existing_value_type is frozenset:
                 settings[key] = frozenset(_as_list(settings.get(key)))  # type: ignore
-            elif existing_value_type == bool:
+            elif existing_value_type is bool:
                 # Only some configuration formats support native boolean values.
                 if not isinstance(value, bool):
                     value = _as_bool(value)

pyproject.toml

@@ -133,6 +133,7 @@ dev = [
     "pytest-benchmark>=3.4.1",
     "pytest-mock>=1.10",
     "requirementslib>=1.5",
+    "ruff>=0.9.6",
     "safety>=2.2.0",
     "stdlibs>=2024.10.21.16",
     "toml>=0.10.2",
@@ -169,6 +170,37 @@ allow_untyped_defs = true
 allow_incomplete_defs = true
 allow_untyped_calls = true
 
+[tool.ruff]
+line-length = 100
+lint.select = [
+    "ASYNC",
+    "B",
+    "C4",
+    "C90",
+    "E",
+    "F",
+    "FLY",
+    "PIE",
+    "S",
+    "W",
+]
+lint.ignore = [
+    "B017",
+    "B028",
+    "B904",
+    "E203",
+    "E501",
+]
+lint.exclude = [ "_vendored" ]
+lint.mccabe.max-complexity = 91  # Default is 10
+
+[tool.ruff.lint.per-file-ignores]
+"isort/hooks.py" = [ "S603" ]
+"isort/settings.py" = [ "S603", "S607" ]
+"tests/*" = [ "S" ]
+"tests/unit/example_crlf_file.py" = [ "F401" ]
+"tests/unit/test_wrap_modes.py" = [ "PIE804" ]
+
 [tool.isort]
 profile = "hug"
 src_paths = ["isort", "tests"]

scripts/build_profile_docs.py

@@ -1,6 +1,6 @@
 #! /bin/env python
 import os
-from typing import Any, Dict, Generator, Iterable, Type
+from typing import Any, Dict
 
 from isort.profiles import profiles
 

scripts/lint.sh

@@ -7,6 +7,7 @@ uv run black --target-version py39 --check .
 uv run isort --profile hug --check --diff isort/ tests/
 uv run isort --profile hug --check --diff example_*/
 uv run --with=Flake8-pyproject flake8 isort/ tests/
+uv run ruff check
  # 51457: https://github.com/tiangolo/typer/discussions/674
  # 72715: https://github.com/timothycrosley/portray/issues/95
 uv run safety check -i 72715 -i 51457 -i 59587

tests/unit/test_exceptions.py

@@ -86,7 +86,7 @@ def setup_class(self):
 
     def test_variables(self):
         assert self.instance.code == "x = ["
-        assert self.instance.original_error == SyntaxError
+        assert self.instance.original_error is SyntaxError
 
 
 class TestLiteralSortTypeMismatch(TestISortError):
@@ -96,8 +96,8 @@ def setup_class(self):
         )
 
     def test_variables(self):
-        assert self.instance.kind == tuple
-        assert self.instance.expected_kind == list
+        assert self.instance.kind is tuple
+        assert self.instance.expected_kind is list
 
 
 class TestAssignmentsFormatMismatch(TestISortError):