Configure ruff beyond the basics

Especially isort, shame it's not part of format but...

Author: masklinn

pyproject.toml

@@ -43,6 +43,17 @@ classifiers = [
         "Programming Language :: Python :: Implementation :: PyPy"
 ]
 
+[tool.ruff.lint]
+select = ["F", "E", "W", "I", "RET", "RUF", "PT"]
+ignore = [
+    "RET505",  # elif after return
+    "E501",  # line too long, formatter should take care of the fixable ones
+]
+
+[tool.ruff.lint.isort]
+classes = ["LRU", "OS"]
+combine-as-imports = true
+
 [tool.mypy]
 python_version = "3.8"
 files = "src,tests"

src/ua_parser/__init__.py

@@ -48,6 +48,8 @@
 import contextlib
 from typing import Callable, Optional
 
+from .basic import Parser as BasicParser
+from .caching import CachingParser, Clearing, Locking, LRU
 from .core import (
     DefaultedParseResult,
     Device,
@@ -56,15 +58,13 @@
     Matchers,
     OS,
     OSMatcher,
-    ParseResult,
     Parser,
+    ParseResult,
     PartialParseResult,
     UserAgent,
     UserAgentMatcher,
 )
-from .basic import Parser as BasicParser
-from .caching import CachingParser, Clearing, LRU, Locking
-from .loaders import load_builtins, load_lazy_builtins, load_data, load_yaml
+from .loaders import load_builtins, load_data, load_lazy_builtins, load_yaml
 
 Re2Parser: Optional[Callable[[Matchers], Parser]] = None
 with contextlib.suppress(ImportError):

src/ua_parser/_lazy.pyi

@@ -1,7 +1,8 @@
 __all__ = ["MATCHERS"]
 
-from typing import Tuple, List
-from .lazy import UserAgentMatcher, OSMatcher, DeviceMatcher
+from typing import List, Tuple
+
+from .lazy import DeviceMatcher, OSMatcher, UserAgentMatcher
 
 MATCHERS: Tuple[
     List[UserAgentMatcher],

src/ua_parser/_matchers.pyi

@@ -1,7 +1,8 @@
 __all__ = ["MATCHERS"]
 
-from typing import Tuple, List
-from .core import UserAgentMatcher, OSMatcher, DeviceMatcher
+from typing import List, Tuple
+
+from .core import DeviceMatcher, OSMatcher, UserAgentMatcher
 
 MATCHERS: Tuple[
     List[UserAgentMatcher],

src/ua_parser/_regexes.pyi

@@ -1,5 +1,6 @@
 from typing import List
-from .user_agent_parser import UserAgentParser, OSParser, DeviceParser
+
+from .user_agent_parser import DeviceParser, OSParser, UserAgentParser
 
 USER_AGENT_PARSERS: List[UserAgentParser]
 OS_PARSERS: List[OSParser]

src/ua_parser/bench.py

@@ -1,25 +1,25 @@
 import argparse
 import csv
-import itertools
 import io
-import time
+import itertools
 import sys
+import time
 from typing import Any, Callable, Iterable, List, Optional
 
 from . import (
-    load_builtins,
-    load_yaml,
     BasicParser,
     CachingParser,
     Clearing,
-    LRU,
     Locking,
+    LRU,
     Matchers,
     Parser,
+    load_builtins,
+    load_yaml,
 )
 from .caching import Cache
-from .user_agent_parser import Parse
 from .re2 import Parser as Re2Parser
+from .user_agent_parser import Parse
 
 CACHEABLE = {
     "basic": True,

src/ua_parser/caching.py

@@ -1,10 +1,9 @@
 import abc
-from collections import OrderedDict
 import threading
+from collections import OrderedDict
 from typing import Dict, Optional
 
-from .core import Parser, Domain, PartialParseResult
-
+from .core import Domain, Parser, PartialParseResult
 
 __all__ = [
     "CachingParser",

src/ua_parser/core.py

@@ -2,7 +2,7 @@
 import re
 from dataclasses import dataclass
 from enum import Flag, auto
-from typing import Generic, Literal, Optional, Tuple, List, TypeVar, Match, Pattern
+from typing import Generic, List, Literal, Match, Optional, Pattern, Tuple, TypeVar
 
 __all__ = [
     "DefaultedParseResult",

src/ua_parser/lazy.py

@@ -4,7 +4,7 @@
 from functools import cached_property
 from typing import Literal, Optional, Pattern
 
-from .core import Matcher, UserAgent, OS, Device, _replacer, _get
+from .core import Device, Matcher, OS, UserAgent, _get, _replacer
 
 
 class UserAgentMatcher(Matcher[UserAgent]):

src/ua_parser/loaders.py

@@ -15,6 +15,7 @@
 import json
 import os
 from typing import (
+    TYPE_CHECKING,
     Callable,
     List,
     Literal,
@@ -24,21 +25,20 @@
     Type,
     TypedDict,
     Union,
-    TYPE_CHECKING,
     cast,
 )
 
 from . import lazy
-from .core import Matchers, UserAgentMatcher, OSMatcher, DeviceMatcher
+from .core import DeviceMatcher, Matchers, OSMatcher, UserAgentMatcher
 
 if TYPE_CHECKING:
     PathOrFile = Union[str, os.PathLike[str], io.IOBase]
     SafeLoader: Optional[Type[object]]
 try:
-    from yaml import load, CSafeLoader as SafeLoader
+    from yaml import CSafeLoader as SafeLoader, load
 except ImportError:
     try:
-        from yaml import load, SafeLoader
+        from yaml import SafeLoader, load
     except ImportError:
         load = SafeLoader = None  # type: ignore