DEV: Remove ignore Ruff rule RET502 (#3208)

RET502: Do not implicitly `return None` in function able to return non-`None` value.

Author: j-t-1

pypdf/_writer.py

@@ -2057,7 +2057,7 @@ def remove_objects_from_page(
         if isinstance(to_delete, (list, tuple)):
             for to_d in to_delete:
                 self.remove_objects_from_page(page, to_d)
-            return
+            return None
         assert isinstance(to_delete, ObjectDeletionFlag)
 
         if to_delete & ObjectDeletionFlag.LINKS:

pyproject.toml

@@ -172,7 +172,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
-    "RET502",  # Do not implicitly `return None` in function able to return non-`None` value
     "RET503",  # Missing explicit `return` at the end of function able to return non-`None` value
     "RET505",  # Unnecessary `else` after `return` statement
     "RUF001",  # Detect confusable Unicode-to-Unicode units. Introduces bugs

tests/__init__.py

@@ -168,5 +168,5 @@ def __exit__(self, type_, value, traceback) -> Optional[bool]:
         ImageFile.LOAD_TRUNCATED_IMAGES = self._saved_load_truncated_images
         if type_:
             # Error.
-            return
+            return None
         return True