DEV: Remove ignore Ruff rule RET503

pypdf/_page.py

@@ -1080,6 +1080,7 @@ def _merge_page(
                 return self._merge_page_writer(
                     page2, page2transformation, ctm, over, expand
                 )
+                return None
         except (AssertionError, AttributeError):
             pass
 
@@ -1172,6 +1173,7 @@ def _merge_page(
         self.replace_contents(ContentStream(new_content_array, self.pdf))
         self[NameObject(PG.RESOURCES)] = new_resources
         self[NameObject(PG.ANNOTS)] = new_annots
+        return None
 
     def _merge_page_writer(
         self,

pypdf/_writer.py

@@ -2232,6 +2232,7 @@ def clean_forms(
 
             clean(content, images, forms, text_filters)
             page.replace_contents(content)
+        return [], []  # type: ignore
 
     def remove_images(
         self,
@@ -3205,6 +3206,7 @@ def find_outline_item(
                 o = cast(TreeObject, o["/Next"])
             else:
                 return None
+        return None  # pragma: no cover
 
     def reset_translation(
         self, reader: Union[None, PdfReader, IndirectObject] = None

pyproject.toml

@@ -179,7 +179,6 @@ ignore = [
     "PT014",   # Ruff bug: Duplicate of test case at index 1 in `@pytest_mark.parametrize`
     "PTH123",  # `open()` should be replaced by `Path.open()`
     "PYI042",  # Type alias `mode_str_type` should be CamelCase
-    "RET503",  # Missing explicit `return` at the end of function able to return non-`None` value
     "RUF001",  # Detect confusable Unicode-to-Unicode units. Introduces bugs
     "RUF002",  # Detect confusable Unicode-to-Unicode units. Introduces bugs
     "S101",    # Use of `assert` detected

tests/test_merger.py

@@ -24,8 +24,15 @@ def merger_operate(merger):
     pdf_forms = RESOURCE_ROOT / "pdflatex-forms.pdf"
     pdf_pw = RESOURCE_ROOT / "libreoffice-writer-password.pdf"
 
-    # string path:
     merger.append(pdf_path)
+
+    # For code coverage
+    found_oi = merger.find_outline_item("nothing here")
+    assert found_oi is None
+
+    found_oi = merger.find_outline_item("nothing here 2", None)
+    assert found_oi is None
+
     merger.append(outline)
     merger.append(pdf_path, pages=pypdf.pagerange.PageRange(slice(0, 0)))
     merger.append(pdf_forms)