chore: ruff fixes

src/jinjarope/environment.py

@@ -181,7 +181,7 @@ def load_jinja_file(
             self._add_loader(loader)
 
     @overload
-    def compile(  # type: ignore[misc]  # noqa: A003
+    def compile(  # type: ignore[misc]
         self,
         source: str | jinja2.nodes.Template,
         name: str | None = None,
@@ -191,7 +191,7 @@ def compile(  # type: ignore[misc]  # noqa: A003
     ) -> CodeType: ...
 
     @overload
-    def compile(  # noqa: A003
+    def compile(
         self,
         source: str | jinja2.nodes.Template,
         name: str | None = None,
@@ -200,7 +200,7 @@ def compile(  # noqa: A003
         defer_init: bool = False,
     ) -> str: ...
 
-    def compile(  # noqa: A003
+    def compile(
         self,
         source: str | jinja2.nodes.Template,
         name: str | None = None,

src/jinjarope/fsspecloaders.py

@@ -31,7 +31,7 @@ class FsSpecProtocolPathLoader(loaders_.LoaderMixin, jinja2.BaseLoader):
     ID = "fsspec_protocol_path"
 
     def __eq__(self, other):
-        return type(self) == type(other)
+        return type(self) is type(other)
 
     def __hash__(self):
         return hash(type(self))
@@ -117,7 +117,7 @@ def __repr__(self):
 
     def __eq__(self, other):
         return (
-            type(self) == type(other)
+            type(self) is type(other)
             and self.storage_options == other.storage_options
             and self.fs == other.fs
             and self.path == other.path

src/jinjarope/htmlfilters.py

@@ -6,7 +6,7 @@
 import posixpath
 import re
 from typing import TYPE_CHECKING, Any, Literal
-from xml.etree import ElementTree as Et
+from xml.etree import ElementTree as ET
 
 
 if TYPE_CHECKING:
@@ -170,7 +170,7 @@ def _parse(obj, selector: str = ""):
 
 @functools.lru_cache
 def format_xml(
-    str_or_elem: str | Et.Element,
+    str_or_elem: str | ET.Element,
     indent: str | int = "  ",
     level: int = 0,
     method: Literal["xml", "html", "text", "c14n"] = "html",
@@ -190,10 +190,10 @@ def format_xml(
                          (applies when mode is "xml")
     """
     if isinstance(str_or_elem, str):
-        str_or_elem = Et.fromstring(str_or_elem)
+        str_or_elem = ET.fromstring(str_or_elem)
     space = indent if isinstance(indent, str) else indent * " "
-    Et.indent(str_or_elem, space=space, level=level)
-    return Et.tostring(
+    ET.indent(str_or_elem, space=space, level=level)
+    return ET.tostring(
         str_or_elem,
         encoding="unicode",
         method=method,

src/jinjarope/loaders.py

@@ -60,7 +60,7 @@ def __repr__(self):
         return utils.get_repr(self, self.mapping)
 
     def __eq__(self, other):
-        return type(self) == type(other) and self.mapping == other.mapping
+        return type(self) is type(other) and self.mapping == other.mapping
 
     def __hash__(self):
         return hash(tuple(sorted(self.mapping.items())))
@@ -85,7 +85,7 @@ def __repr__(self):
 
     def __eq__(self, other):
         return (
-            type(self) == type(other)
+            type(self) is type(other)
             and self.package_name == other.package_name
             and self.module == other.module
         )
@@ -107,7 +107,7 @@ def __repr__(self):
         return utils.get_repr(self, self.load_func)
 
     def __eq__(self, other):
-        return type(self) == type(other) and self.load_func == other.load_func
+        return type(self) is type(other) and self.load_func == other.load_func
 
     def __hash__(self):
         return hash(self.load_func)
@@ -151,7 +151,7 @@ def __repr__(self):
 
     def __eq__(self, other):
         return (
-            type(self) == type(other)
+            type(self) is type(other)
             and self.package_name == other.package_name
             and self.package_path == other.package_path
         )
@@ -176,7 +176,7 @@ def __bool__(self):
         return len(self.searchpath) > 0
 
     def __eq__(self, other):
-        return type(self) == type(other) and self.searchpath == other.searchpath
+        return type(self) is type(other) and self.searchpath == other.searchpath
 
     def __hash__(self):
         return hash(tuple(self.searchpath))
@@ -194,7 +194,7 @@ def __bool__(self):
         return len(self.loaders) > 0
 
     def __eq__(self, other):
-        return type(self) == type(other) and self.loaders == other.loaders
+        return type(self) is type(other) and self.loaders == other.loaders
 
     def __hash__(self):
         return hash(tuple(self.loaders))
@@ -219,7 +219,7 @@ def __add__(self, other):
         return DictLoader(mapping)
 
     def __eq__(self, other):
-        return type(self) == type(other) and self.mapping == other.mapping
+        return type(self) is type(other) and self.mapping == other.mapping
 
     def __hash__(self):
         return hash(tuple(sorted(self.mapping.items())))

src/jinjarope/regexfilters.py

@@ -120,4 +120,5 @@ def re_escape(string: str, re_type: Literal["python", "posix_basic"] = "python")
             # list of BRE special chars:
             return re_replace(string, r"([].[^$*\\])", r"\\\1")
         case _:
-            raise NotImplementedError("Invalid regex type (%s)" % re_type)
+            msg = f"Invalid regex type ({re_type})"
+            raise NotImplementedError(msg)

src/jinjarope/rewriteloader.py

@@ -39,7 +39,7 @@ def __repr__(self):
 
     def __eq__(self, other):
         return (
-            type(self) == type(other)
+            type(self) is type(other)
             and self.loader == other.loader
             and self.rewrite_fn == other.rewrite_fn
         )

tests/test_cli.py

@@ -1,8 +1,9 @@
 from __future__ import annotations
 
-from jinjarope import cli
 from typer.testing import CliRunner
 
+from jinjarope import cli
+
 
 def test_render_with_cli():
     runner = CliRunner()

tests/test_environment.py

@@ -3,9 +3,10 @@
 from typing import TYPE_CHECKING
 
 import jinja2
-import jinjarope
 import pytest
 
+import jinjarope
+
 
 if TYPE_CHECKING:
     import pathlib

tests/test_fsspec_loaders.py

@@ -1,9 +1,10 @@
 from __future__ import annotations
 
 import jinja2
-from jinjarope import environment, fsspecloaders
 import pytest
 
+from jinjarope import environment, fsspecloaders
+
 
 def test_fsspec_protocol_loader():
     env = environment.Environment()

tests/test_htmlfilters.py

@@ -1,10 +1,11 @@
 from __future__ import annotations
 
-from xml.etree import ElementTree as Et
+from xml.etree import ElementTree as ET
 
-from jinjarope import htmlfilters
 import pytest
 
+from jinjarope import htmlfilters
+
 
 def test_wrap_in_elem():
     assert htmlfilters.wrap_in_elem("Hello", "p") == "<p>Hello</p>"
@@ -92,8 +93,8 @@ def test_format_xml_with_str():
 
 
 def test_format_xml_with_element():
-    xml_elem = Et.Element("root")
-    child = Et.SubElement(xml_elem, "child")
+    xml_elem = ET.Element("root")
+    child = ET.SubElement(xml_elem, "child")
     child.text = "text"
     formatted_xml = htmlfilters.format_xml(xml_elem)
     expected_xml = "<root>\n  <child>text</child>\n</root>"

tests/test_jinjaloaderfilesystem.py

@@ -1,9 +1,10 @@
 from __future__ import annotations
 
 import jinja2
-from jinjarope import jinjaloaderfilesystem
 import pytest
 
+from jinjarope import jinjaloaderfilesystem
+
 
 dct = {"home.html": "Home", "about.html": "About", "subfolder/sub.html": "Sub"}
 

tests/test_manual.py

@@ -1,8 +1,9 @@
 from __future__ import annotations
 
-from jinjarope import manual
 import mknodes as mk
 
+from jinjarope import manual
+
 
 def test_building_the_docs():
     build = manual.Build()

tests/test_nested_dict_loader.py

@@ -1,9 +1,10 @@
 from __future__ import annotations
 
 import jinja2
-from jinjarope import configloaders
 import pytest
 
+from jinjarope import configloaders
+
 
 def test_nested_dict_loader():
     templates = {

tests/test_regexfilters.py

@@ -1,9 +1,10 @@
 from __future__ import annotations
 
 from jinja2.exceptions import FilterArgumentError
-from jinjarope import regexfilters
 import pytest
 
+from jinjarope import regexfilters
+
 
 def test_re_replace():
     assert (

tests/test_template_file_loader.py

@@ -4,9 +4,10 @@
 from typing import TYPE_CHECKING
 
 import jinja2
-from jinjarope import configloaders
 import pytest
 
+from jinjarope import configloaders
+
 
 if TYPE_CHECKING:
     import pathlib