Switch to Ruff for linting

Author: oschwald

.gitignore

@@ -67,9 +67,7 @@ MANIFEST
 .project
 .pydevproject
 .python-version
-pylint.txt
 *.swp
-violations.pyflakes.txt
 
 /t.py
 /test*.py

minfraud/models.py

@@ -2,7 +2,6 @@
 
 from __future__ import annotations
 
-# pylint:disable=too-many-lines,too-many-instance-attributes,too-many-locals
 from typing import TYPE_CHECKING
 
 import geoip2.models
@@ -232,7 +231,6 @@ def __init__(
         self,
         *,
         confidence: float | None = None,
-        # pylint:disable=redefined-builtin
         id: str | None = None,
         last_seen: str | None = None,
         local_time: str | None = None,
@@ -382,7 +380,6 @@ def __init__(
         is_issued_in_billing_address_country: bool | None = None,
         is_prepaid: bool | None = None,
         is_virtual: bool | None = None,
-        # pylint:disable=redefined-builtin
         type: str | None = None,  # noqa: A002
     ) -> None:
         """Initialize a CreditCard instance."""
@@ -752,7 +749,7 @@ class Reason(_Serializable):
 
     See the `risk reasons documentation <https://dev.maxmind.com/minfraud/api-documentation/responses/#schema--response--risk-score-reason--multiplier-reason>`_
     for the current list of reason codes.
-    """  # pylint:disable=line-too-long
+    """
 
     code: str | None
     """This value is a machine-readable code identifying the reason.
@@ -953,7 +950,6 @@ def __init__(
         funds_remaining: float,
         device: dict | None = None,
         email: dict | None = None,
-        # pylint:disable=redefined-builtin
         id: str,
         ip_address: dict | None = None,
         queries_remaining: int,
@@ -1062,7 +1058,6 @@ def __init__(
         disposition: dict | None = None,
         email: dict | None = None,
         funds_remaining: float,
-        # pylint:disable=redefined-builtin
         id: str,
         ip_address: dict | None = None,
         queries_remaining: int,
@@ -1130,7 +1125,6 @@ def __init__(
         *,
         disposition: dict | None = None,
         funds_remaining: float,
-        # pylint:disable=redefined-builtin
         id: str,
         ip_address: dict | None = None,
         queries_remaining: int,

minfraud/validation.py

@@ -29,12 +29,6 @@
 )
 from voluptuous.error import UrlInvalid
 
-# Pylint doesn't like the private function type naming for the callable
-# objects below. Given the consistent use of them, the current names seem
-# preferable to blindly following pylint.
-#
-# pylint: disable=invalid-name,undefined-variable
-
 _any_number = Any(float, int, Decimal)
 
 _custom_input_key = All(str, Match(r"^[a-z0-9_]{1,25}$"))

minfraud/webservice.py

@@ -42,7 +42,6 @@
 _SCHEME = "https"
 
 
-# pylint: disable=too-many-instance-attributes
 class BaseClient:
     """Base class for minFraud clients."""
 

pyproject.toml

@@ -38,9 +38,7 @@ dev = [
     "pytest-httpserver>=1.0.10",
 ]
 lint = [
-    "flake8>=7.2.0",
     "mypy>=1.15.0",
-    "pylint>=3.3.6",
     "ruff>=0.11.6",
     "types-requests>=2.32.0.20250328",
 ]
@@ -122,9 +120,8 @@ dependency_groups = [
     "lint",
 ]
 commands = [
-    ["flake8", "minfraud"],
     ["mypy", "minfraud", "tests"],
-    ["pylint", "minfraud"],
+    ["ruff", "check"],
     ["ruff", "format", "--check", "--diff", "."],
 ]