Update ruff settings; update hooks (#53)

.pre-commit-config.yaml

@@ -1,14 +1,14 @@
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.6.0
+    rev: v5.0.0
     hooks:
       - id: check-toml
       - id: check-yaml
       - id: end-of-file-fixer
       - id: trailing-whitespace
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.5.7
+    rev: v0.7.1
     hooks:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix, --show-fixes]
@@ -20,8 +20,14 @@ repos:
       - id: numpydoc-validation
         exclude: (tests/|docs/).*
 
+  - repo: https://github.com/econchick/interrogate
+    rev: 1.7.0
+    hooks:
+      - id: interrogate
+        files: tests/.*
+
   - repo: https://github.com/tox-dev/pyproject-fmt
-    rev: 2.2.1
+    rev: v2.4.3
     hooks:
       - id: pyproject-fmt
         args: [--keep-full-version, --no-print-diff]

README.md

@@ -42,7 +42,7 @@
      </td>
    </tr>
   </table>
-  
+
   <hr>
 </div>
 
@@ -58,7 +58,7 @@ Add the following to your `.pre-commit-config.yaml` file:
 
 ```yaml
 - repo: https://github.com/stefmolin/exif-stripper
-  rev: 0.4.0
+  rev: 0.5.0
   hooks:
     - id: strip-exif
 ```

pyproject.toml

@@ -63,19 +63,28 @@ version = { attr = "exif_stripper.__version__" }
 line-length = 88
 format.indent-style = "space"
 format.quote-style = "single"
+format.docstring-code-format = true
 lint.select = [
-  "B", # flake8-bugbear rules
-  "C", # mccabe rules
-  "E", # pycodestyle error rules
-  "F", # pyflakes rules
-  "I", # isort rules
-  "W", # pycodestyle warning rules
+  "ANN", # flake8-annotations
+  "B",   # flake8-bugbear
+  "C4",  # flake8-comprehensions
+  "E",   # pycodestyle error
+  "ERA", # eradicate (commented out code)
+  "F",   # pyflakes
+  "FA",  # flake8-future-annotations
+  "I",   # isort
+  "N",   # pep8-naming
+  "PTH", # flake8-use-pathlib
+  "SIM", # flake8-simplify
+  "TRY", # tryceratops
+  "UP",  # pyupgrade
+  "W",   # pycodestyle warning
 ]
 lint.ignore = [
-  "C901", # max-complexity-10
   "E501", # line-too-long
 ]
 
+lint.extend-per-file-ignores."tests/*" = [ "ANN" ] # don't require annotations for tests
 lint.isort.known-first-party = [
   "exif_stripper",
 ]

src/exif_stripper/__init__.py

@@ -7,7 +7,7 @@
 
 from PIL import Image, UnidentifiedImageError
 
-__version__ = '0.4.0'
+__version__ = '0.5.0'
 
 
 def process_image(filename: str | os.PathLike) -> bool:

tests/test_cli.py

@@ -3,6 +3,7 @@
 import platform
 import subprocess
 import sys
+from contextlib import suppress
 from getpass import getuser
 
 import pytest
@@ -33,16 +34,14 @@ def image_with_exif_data(tmp_path):
 def image_with_metadata(image_with_exif_data):
     """Fixture for an image with metadata."""
     if RUNNING_ON in ['Darwin', 'Linux']:
-        try:
+        with suppress(OSError):
+            # OSError raised if filesystem does not support extended attributes
             xattr(image_with_exif_data).set(
                 f'{getuser()}.test_extended_attribute'
                 if RUNNING_ON == 'Linux'
                 else 'com.apple.macl',
                 b'\x00',
             )
-        except OSError:  # pragma: nocover
-            # filesystem does not support extended attributes
-            pass
     return image_with_exif_data