Replace flake8 with ruff linter

.github/workflows/validation.yml

@@ -39,7 +39,7 @@ jobs:
           restore-keys: "precommit-${{ runner.os }}-${{ steps.poetry_setup.outputs-python-version}}-"
 
       - name: Run pre-commit hooks
-        run: SKIP=black,isort,flake8,pyright pre-commit run --all-files
+        run: SKIP=black,isort,ruff,pyright pre-commit run --all-files
 
       - name: Run black formatter check
         run: black --check --diff .
@@ -50,8 +50,8 @@ jobs:
       - name: Run pyright type checker
         run: pyright .
 
-      - name: Run flake8 linter
-        run: flake8 .
+      - name: Run ruff linter
+        run: ruff check --show-source .
 
       # Steps below are here to generate and upload an artifact from
       # this workflow so that we can have the data about author and some

.pre-commit-config.yaml

@@ -26,12 +26,14 @@ repos:
 
   - repo: local
     hooks:
-      - id: flake8
-        name: Flake8
-        description: Run flake8 checks on the code
-        entry: poetry run flake8
+      - id: ruff
+        name: Ruff
+        description: Run ruff checks on the code
+        entry: poetry run ruff check --force-exclude
         language: system
         types: [python]
+        require_serial: true
+        args: [--fix, --exit-non-zero-on-fix]
 
   - repo: local
     hooks:

mcstatus/__init__.py

@@ -1,4 +1,4 @@
-from mcstatus.server import BedrockServer, JavaServer, MCServer  # noqa: F401
+from mcstatus.server import BedrockServer, JavaServer, MCServer
 
 __all__ = [
     "MCServer",

mcstatus/utils.py

@@ -124,7 +124,7 @@ def deprecated(
 
 
 def deprecated(
-    obj: Any = None,  # noqa: ANN401
+    obj: Any = None,
     *,
     replacement: str | None = None,
     version: str | None = None,

pyproject.toml

@@ -46,17 +46,10 @@ pytest-rerunfailures = "^11.1.2"
 coverage = "^7.2.6"
 
 [tool.poetry.group.lint.dependencies]
+ruff = "^0.0.278"
 black = "^23.3.0"
 coverage = "^7.2.6"
-flake8 = "^6.0.0"
-flake8-annotations = "^3.0.1"
-flake8-bugbear = "^23.5.9"
-flake8-tidy-imports = "^4.8.0"
-flake8-pep585 = {version = "^0.1.7", python = ">=3.9"}
-flake8-new-union-types = "^0.4.1"
-flake8-future-annotations = "^1.1.0"
 isort = "^5.12.0"
-pep8-naming = "^0.13.3"
 pyright = "^1.1.309"
 typing-extensions = "^4.6.1"
 
@@ -99,6 +92,44 @@ bump = true
 enable = true
 style = "pep440"
 
+[tool.ruff]
+target-version = "py38"
+line-length = 127
+
+select = [
+    "F",     # Pyflakes
+    "W",     # Pycodestyle (warnigns)
+    "E",     # Pycodestyle (errors)
+    "N",     # pep8-naming
+    "ANN",   # flake8-annotations
+    "B",     # flake8-bugbear
+    "FA",    # flake8-future-annotations
+    "TID",   # flake8-tidy-imports
+    "UP007", # flake8-new-union-types equivalent rule
+    "RUF",   # ruff's custom rules
+]
+
+ignore = [
+    "ANN002", # Missing type annotation for *args
+    "ANN003", # Missing type annotation for **kwargs
+    "ANN101", # Missing type annotation for self in method
+    "ANN102", # Missing type annotation for cls in classmethod
+    "ANN204", # Missing return type annotation for special method
+    "B904",   # Exception raised within try-except should use raise ... from exc
+    "RUF012", # Mutable class attributes should be annotated with `typing.ClassVar`
+]
+
+[tool.ruff.extend-per-file-ignores]
+"test_*.py" = [
+    "ANN", # flake8-annotations
+]
+"docs/examples/code/**" = [
+    "FA",  # flake8-future-annotations
+]
+
+[tool.ruff.flake8-tidy-imports]
+ban-relative-imports = "all"
+
 [tool.black]
 line-length = 127
 

tests/motd/test_transformers.py

@@ -71,8 +71,8 @@ def test_correct_output(self, result: Callable[["str | dict", bool], str], sourc
             "<span style='color:rgb(170, 0, 0);text-shadow:0 0 1px rgb(42, 0, 0)'>8</span>"
             "<span style='color:rgb(170, 0, 170);text-shadow:0 0 1px rgb(42, 0, 42)'>9</span>"
             "<span style='color:rgb(255, 170, 0);text-shadow:0 0 1px rgb"
-            + str((64, 42, 0) if bedrock else (42, 42, 0))  # noqa: W503
-            + "'>10</span>"  # noqa: W503
+            + str((64, 42, 0) if bedrock else (42, 42, 0))
+            + "'>10</span>"
             "<span style='color:rgb(170, 170, 170);text-shadow:0 0 1px rgb(42, 42, 42)'>11</span>"
             "<span style='color:rgb(85, 85, 85);text-shadow:0 0 1px rgb(21, 21, 21)'>12</span>"
             "<span style='color:rgb(85, 85, 255);text-shadow:0 0 1px rgb(21, 21, 63)'>13</span>"

tests/status_response/__init__.py

@@ -16,7 +16,7 @@ class BaseStatusResponseTest(abc.ABC):
     # if we don't specify item in raw answer, target field will be None
     # first element is a list with fields to remove, and attribute that
     # must be None. a dict is a raw answer to pass into `build` method
-    OPTIONAL_FIELDS: tuple[list[tuple[str, str]], dict[str, Any]] | None = None  # noqa: ANN401
+    OPTIONAL_FIELDS: tuple[list[tuple[str, str]], dict[str, Any]] | None = None
 
     def _validate(self) -> None:
         """Perform checks to validate the class."""