refactor: switch lint environment to python 3.10 and corrected all linting errors and warnings

d-biehl · d-biehl · commit 19a26325454b · 2025-06-20T19:16:04.000+02:00
diff --git a/bundled_requirements.txt b/bundled_requirements.txt
@@ -4,5 +4,4 @@ pluggy>=1.0.0
 tomli>=2.0.0
 tomli_w>=1.0.0
 platformdirs>=3.2.0,<4.4.0
-colorama>=0.4.6
-websockets>=15.0
+colorama>=0.4.6
diff --git a/packages/core/src/robotcode/core/event.py b/packages/core/src/robotcode/core/event.py
@@ -72,10 +72,10 @@ def __iter__(self) -> Iterator[Callable[_TParams, _TResult]]:
 
     def _notify(
         self,
-        *__args: _TParams.args,
+        *__args: Any,
         return_exceptions: Optional[bool] = True,
         callback_filter: Optional[Callable[[Callable[..., Any]], bool]] = None,
-        **__kwargs: _TParams.kwargs,
+        **__kwargs: Any,
     ) -> Iterator[Union[_TResult, BaseException]]:
         for method in filter(
             lambda x: callback_filter(x) if callback_filter is not None else True,
@@ -95,19 +95,19 @@ def _notify(
 class EventIterator(EventResultIteratorBase[_TParams, _TResult]):
     def __call__(
         self,
-        *__args: _TParams.args,
+        *__args: Any,
         callback_filter: Optional[Callable[[Callable[..., Any]], bool]] = None,
-        **__kwargs: _TParams.kwargs,
+        **__kwargs: Any,
     ) -> Iterator[Union[_TResult, BaseException]]:
         return self._notify(*__args, callback_filter=callback_filter, **__kwargs)
 
 
 class Event(EventResultIteratorBase[_TParams, _TResult]):
     def __call__(
         self,
-        *__args: _TParams.args,
+        *__args: Any,
         callback_filter: Optional[Callable[[Callable[..., Any]], bool]] = None,
-        **__kwargs: _TParams.kwargs,
+        **__kwargs: Any,
     ) -> List[Union[_TResult, BaseException]]:
         return list(self._notify(*__args, callback_filter=callback_filter, **__kwargs))
 
diff --git a/packages/core/src/robotcode/core/text_document.py b/packages/core/src/robotcode/core/text_document.py
@@ -226,7 +226,7 @@ def cache_invalidated(sender) -> None: ...
     def _cache_invalidating(self) -> Iterator[None]:
         send_events = len(self._cache) > 0
         if send_events:
-            self.cache_invalidate()
+            self.cache_invalidate(self)
         try:
             with self._lock:
                 yield
diff --git a/packages/core/src/robotcode/core/utils/dataclasses.py b/packages/core/src/robotcode/core/utils/dataclasses.py
@@ -647,7 +647,7 @@ def validate_types(expected_types: Union[type, Tuple[type, ...], None], value: A
             continue
 
         if (
-            t is Any  # type: ignore
+            t is Any
             or t is Ellipsis  # type: ignore
             or isinstance(value, origin or t)
             or (
@@ -683,7 +683,7 @@ def validate_types(expected_types: Union[type, Tuple[type, ...], None], value: A
 
                 return []
 
-            if t is Any:  # type: ignore
+            if t is Any:
                 return []
 
             if isinstance(value, origin or t):
diff --git a/packages/core/src/robotcode/core/utils/inspect.py b/packages/core/src/robotcode/core/utils/inspect.py
@@ -15,7 +15,7 @@ def iter_methods(
                 m = v
             else:
                 m = getattr(obj, name)
-                if not inspect.ismethod(m):  # type: ignore
+                if not inspect.ismethod(m):
                     continue
 
             if predicate is None or predicate(m):
diff --git a/packages/core/src/robotcode/core/utils/logging.py b/packages/core/src/robotcode/core/utils/logging.py
@@ -136,7 +136,7 @@ def __init_logger(self) -> LoggingDescriptor:
 
             if self.__func is not None:
                 if isinstance(self.__func, staticmethod):
-                    returned_logger = self.__func.__func__()  # type: ignore
+                    returned_logger = self.__func.__func__()
                 else:
                     returned_logger = self.__func()
 
diff --git a/packages/core/src/robotcode/core/workspace.py b/packages/core/src/robotcode/core/workspace.py
@@ -76,7 +76,7 @@ def get_configuration(
         section: Type[TConfig],
         scope_uri: Union[str, Uri, None] = None,
     ) -> TConfig:
-        result = self.settings
+        result: Any = self.settings
         for sub_key in str(section).split("."):
             if sub_key in result:
                 result = result.get(sub_key, None)
diff --git a/packages/debugger/src/robotcode/debugger/dap_types.py b/packages/debugger/src/robotcode/debugger/dap_types.py
@@ -801,7 +801,6 @@ class PauseResponse(Response):
     pass
 
 
-@dataclass
 class SteppingGranularity(Enum):
     STATEMENT = "statement"
     LINE = "line"
diff --git a/packages/jsonrpc2/src/robotcode/jsonrpc2/protocol.py b/packages/jsonrpc2/src/robotcode/jsonrpc2/protocol.py
@@ -196,7 +196,7 @@ def _decorator(func: _F) -> Callable[[_F], _F]:
         if isinstance(func, classmethod):
             f = func.__func__  # type: ignore
         elif isinstance(func, staticmethod):
-            f = func.__func__  # type: ignore
+            f = func.__func__
         else:
             f = func
 
diff --git a/packages/jsonrpc2/src/robotcode/jsonrpc2/server.py b/packages/jsonrpc2/src/robotcode/jsonrpc2/server.py
@@ -6,6 +6,7 @@
 from concurrent.futures import ThreadPoolExecutor
 from types import TracebackType
 from typing import (
+    Any,
     BinaryIO,
     Callable,
     Coroutine,
@@ -44,7 +45,7 @@ def close(self) -> None:
         self.rfile.close()
         self.wfile.close()
 
-    def write(self, data: bytes) -> None:
+    def write(self, data: "bytes | bytearray | memoryview[Any]") -> None:
         self.wfile.write(data)
         self.wfile.flush()
 
diff --git a/packages/language_server/src/robotcode/language_server/common/parts/workspace.py b/packages/language_server/src/robotcode/language_server/common/parts/workspace.py
@@ -315,7 +315,7 @@ def get_configuration_raw(
                 List[Any],
             )
 
-        result = self.settings
+        result: Any = self.settings
         for sub_key in str(section).split("."):
             if sub_key in result:
                 result = result.get(sub_key, None)
diff --git a/packages/language_server/src/robotcode/language_server/robotframework/parts/hover.py b/packages/language_server/src/robotcode/language_server/robotframework/parts/hover.py
@@ -73,7 +73,7 @@ def _find_method(self, cls: Type[Any]) -> Optional[_HoverMethod]:
         for base in cls.__bases__:
             method = self._find_method(base)
             if method:
-                return cast(_HoverMethod, method)
+                return method
 
         return None
 
diff --git a/packages/language_server/src/robotcode/language_server/robotframework/parts/inlay_hint.py b/packages/language_server/src/robotcode/language_server/robotframework/parts/inlay_hint.py
@@ -61,7 +61,7 @@ def _find_method(self, cls: Type[Any]) -> Optional[_HandlerMethod]:
         for base in cls.__bases__:
             method = self._find_method(base)
             if method:
-                return cast(_HandlerMethod, method)
+                return method
 
         return None
 
diff --git a/packages/language_server/src/robotcode/language_server/robotframework/parts/references.py b/packages/language_server/src/robotcode/language_server/robotframework/parts/references.py
@@ -113,7 +113,7 @@ def _find_method(self, cls: Type[Any]) -> Optional[_ReferencesMethod]:
         for base in cls.__bases__:
             method = self._find_method(base)
             if method:
-                return cast(_ReferencesMethod, method)
+                return method
         return None
 
     @language_id("robotframework")
diff --git a/packages/language_server/src/robotcode/language_server/robotframework/parts/signature_help.py b/packages/language_server/src/robotcode/language_server/robotframework/parts/signature_help.py
@@ -68,7 +68,7 @@ def _find_method(self, cls: Type[Any]) -> Optional[_SignatureHelpMethod]:
         for base in cls.__bases__:
             method = self._find_method(base)
             if method:
-                return cast(_SignatureHelpMethod, method)
+                return method
         return None
 
     @language_id("robotframework")
diff --git a/packages/plugin/src/robotcode/plugin/click_helper/types.py b/packages/plugin/src/robotcode/plugin/click_helper/types.py
@@ -20,7 +20,7 @@
 T = TypeVar("T", bound=Enum)
 
 
-class EnumChoice(click.Choice, Generic[T]):
+class EnumChoice(click.Choice, Generic[T]):  # type: ignore[type-arg]
     """A click.Choice that accepts Enum values."""
 
     def __init__(
@@ -30,7 +30,7 @@ def __init__(
         excluded: Optional[Set[T]] = None,
     ) -> None:
         super().__init__(
-            choices if excluded is None else (set(choices).difference(excluded)),  # type: ignore
+            choices if excluded is None else (set(choices).difference(excluded)),
             case_sensitive,
         )
 
diff --git a/packages/robot/src/robotcode/robot/diagnostics/library_doc.py b/packages/robot/src/robotcode/robot/diagnostics/library_doc.py
@@ -1328,7 +1328,7 @@ def get_module_spec(module_name: str) -> Optional[ModuleSpec]:
                 member_name = m + "." + member_name if m and member_name is not None else m
 
     if result is not None:
-        return ModuleSpec(  # type: ignore
+        return ModuleSpec(
             name=result.name,
             origin=result.origin,
             submodule_search_locations=(
diff --git a/pyproject.toml b/pyproject.toml
@@ -159,7 +159,7 @@ fail_under = 40
 
 [tool.ruff]
 line-length = 120
-target-version = "py38"
+target-version = "py310"
 extend-exclude = ["bundled/libs", ".hatch"]
 
 [tool.ruff.lint]
@@ -216,7 +216,7 @@ mark-parentheses = false
 
 
 [tool.mypy]
-python_version = "3.8"
+python_version = "3.10"
 strict = true
 warn_redundant_casts = true
 warn_unused_ignores = true
diff --git a/scripts/generate_rf_options.py b/scripts/generate_rf_options.py
@@ -96,7 +96,8 @@ def generate(
                 current_line = ""
             m = RE_LIST_MATCHER.match(line)
             if not in_examples and m:
-                line = f"**{m.group(1)}{':\\\\' + m.group(4) if m.group(4) else ''}:** {m.group(5)}"
+                p = ":\\\\"
+                line = f"**{m.group(1)}{p + m.group(4) if m.group(4) else ''}:** {m.group(5)}"
                 current_line += "\n\n" + line.strip()
             else:
                 current_line += "\n" + line.strip()
