Lint

.github/workflows/ruff.yml

@@ -16,8 +16,8 @@ jobs:
       - uses: actions/checkout@v4
 
       # keep in sync with .pre-commit-config.yaml
-      - run: python -Im pip install --user ruff==0.4.8
+      - run: python -Im pip install --user ruff==0.6.1
 
       - name: Run ruff
         working-directory: .
-        run: ruff --output-format=github wagtail_localize
+        run: ruff check --output-format=github wagtail_localize

.pre-commit-config.yaml

@@ -14,7 +14,7 @@ repos:
       - id: trailing-whitespace
   - repo: https://github.com/astral-sh/ruff-pre-commit
     # keep in sync with .github/workflows/ruff.yml
-    rev: 'v0.4.8'
+    rev: 'v0.6.1'
     hooks:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix]

wagtail_localize/strings.py

@@ -335,12 +335,9 @@ def ignore_if_at_end(element):
 
             # Ignore if there are no text nodes
             # This will exclude both <br> tags and empty inline tags
-            if not any(
+            return not any(
                 isinstance(desc, NavigableString) for desc in element.descendants
-            ):
-                return True
-
-            return False
+            )
 
         if ignore_if_at_end(elements[0]):
             wrap(elements[1:])