chore: update pre-commit hooks

.pre-commit-config.yaml

@@ -1,7 +1,7 @@
 # https://pre-commit.com
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.3.0
+    rev: v4.6.0
     hooks:
       - id: check-added-large-files
       - id: check-case-conflict
@@ -24,7 +24,7 @@ repos:
           - --markdown-linebreak-ext=md
 
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v0.971
+    rev: v1.10.0
     hooks:
       - id: mypy
         args:
@@ -33,14 +33,14 @@ repos:
           - types-requests
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.0.291
+    rev: v0.4.4
     hooks:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix]
       - id: ruff-format
 
   - repo: https://github.com/shellcheck-py/shellcheck-py
-    rev: v0.8.0.4
+    rev: v0.10.0.1
     hooks:
       - id: shellcheck
 

freeze.py

@@ -1,4 +1,4 @@
-""" Package freezing for Windows. """
+"""Package freezing for Windows."""
 
 import os
 import re

sacad/__init__.py

@@ -1,6 +1,6 @@
 #!/usr/bin/env python3
 
-""" Smart Automatic Cover Art Downloader : search and download music album covers. """
+"""Smart Automatic Cover Art Downloader : search and download music album covers."""
 
 __version__ = "2.7.5"
 __author__ = "desbma"

sacad/__main__.py

@@ -1,6 +1,6 @@
 #!/usr/bin/env python3
 
-""" Command line entry point for sacad program. """
+"""Command line entry point for sacad program."""
 
 import sacad
 

sacad/colored_logging.py

@@ -1,4 +1,4 @@
-""" Formatter for the logging module, coloring terminal output according to error criticity. """
+"""Formatter for the logging module, coloring terminal output according to error criticity."""
 
 import enum
 import logging
@@ -11,7 +11,6 @@
 
 
 class ColoredFormatter(logging.Formatter):
-
     """Logging formatter coloring terminal output according to error criticity."""
 
     def format(self, record):

sacad/cover.py

@@ -1,4 +1,4 @@
-""" Sacad album cover. """
+"""Sacad album cover."""
 
 import asyncio
 import enum
@@ -35,7 +35,6 @@
 
 
 class CoverSourceQuality(enum.IntFlag):
-
     """Flags to describe cover source quality."""
 
     # whether or not the search query matching is fuzzy (does a typo return results ?)
@@ -61,7 +60,6 @@ def isReference(self) -> bool:
 
 
 class CoverImageMetadata(enum.IntFlag):
-
     """Flags to describe image metadata."""
 
     NONE = 0
@@ -83,7 +81,6 @@ def is_square(x):
 
 
 class CoverSourceResult:
-
     """Cover image returned by a source, candidate to be downloaded."""
 
     METADATA_PEEK_SIZE_INCREMENT = 2**12

sacad/http_helpers.py

@@ -1,4 +1,4 @@
-""" Common HTTP code. """
+"""Common HTTP code."""
 
 import asyncio
 import logging
@@ -26,7 +26,6 @@ def aiohttp_socket_timeout(socket_timeout_s):
 
 
 class Http:
-
     """Async HTTP client code."""
 
     def __init__(

sacad/mkstemp_ctx.py

@@ -1,4 +1,4 @@
-""" Additions to the tempfile module. """
+"""Additions to the tempfile module."""
 
 import contextlib
 import os

sacad/rate_watcher.py

@@ -1,4 +1,4 @@
-""" Provide a class with a context manager to help avoid overloading web servers. """
+"""Provide a class with a context manager to help avoid overloading web servers."""
 
 import asyncio
 import logging
@@ -10,7 +10,6 @@
 
 
 class AccessRateWatcher:
-
     """Access rate limiter, supporting concurrent access by threads and/or processes."""
 
     def __init__(

sacad/recurse.py

@@ -1,6 +1,6 @@
 #!/usr/bin/env python3
 
-""" Recursively search and download album covers for a music library. """
+"""Recursively search and download album covers for a music library."""
 
 import argparse
 import asyncio
@@ -36,7 +36,6 @@
 
 # TODO use a dataclasses.dataclass when Python < 3.7 is dropped
 class Work:
-
     """Represent a single search & download work item."""
 
     def __init__(self, cover_filepath, audio_filepaths, metadata):

sacad/sources/__init__.py

@@ -1,4 +1,4 @@
-""" SACAD cover sources. """
+"""SACAD cover sources."""
 
 from sacad.sources.amazoncd import AmazonCdCoverSource, AmazonCdCoverSourceResult  # noqa: F401
 from sacad.sources.amazondigital import AmazonDigitalCoverSource, AmazonDigitalCoverSourceResult  # noqa: F401

sacad/sources/amazonbase.py

@@ -1,10 +1,9 @@
-""" Base class for Amazon cover sources. """
+"""Base class for Amazon cover sources."""
 
 from sacad.sources.base import CoverSource
 
 
 class AmazonBaseCoverSource(CoverSource):
-
     """Base class for Amazon cover sources."""
 
     def __init__(self, *args, base_domain, **kwargs):

sacad/sources/amazoncd.py

@@ -1,4 +1,4 @@
-""" Amazon CD cover source. """
+"""Amazon CD cover source."""
 
 import collections
 import urllib.parse
@@ -11,7 +11,6 @@
 
 
 class AmazonCdCoverSourceResult(CoverSourceResult):
-
     """Amazon CD cover search result."""
 
     def __init__(self, *args, **kwargs):
@@ -21,7 +20,6 @@ def __init__(self, *args, **kwargs):
 
 
 class AmazonCdCoverSource(AmazonBaseCoverSource):
-
     """Cover source returning Amazon.com audio CD images."""
 
     TLDS = ("com", "ca", "cn", "fr", "de", "co.jp", "co.uk")

sacad/sources/amazondigital.py

@@ -1,4 +1,4 @@
-""" Amazon digital cover source. """
+"""Amazon digital cover source."""
 
 import collections
 import operator
@@ -36,7 +36,6 @@
 
 
 class AmazonDigitalCoverSourceResult(CoverSourceResult):
-
     """Amazon digital cover search result."""
 
     def __init__(self, *args, **kwargs):
@@ -46,7 +45,6 @@ def __init__(self, *args, **kwargs):
 
 
 class AmazonDigitalCoverSource(AmazonBaseCoverSource):
-
     """Cover source returning Amazon.com digital music images."""
 
     BASE_URL = "https://www.amazon.com"

sacad/sources/base.py

@@ -1,4 +1,4 @@
-""" Common code for all cover sources cover sources. """
+"""Common code for all cover sources cover sources."""
 
 import abc
 import asyncio
@@ -21,7 +21,6 @@
 
 
 class CoverSource(metaclass=abc.ABCMeta):
-
     """Base class for all cover sources."""
 
     def __init__(

sacad/sources/deezer.py

@@ -1,4 +1,4 @@
-""" Deezer cover source. """
+"""Deezer cover source."""
 
 import collections
 import json
@@ -9,7 +9,6 @@
 
 
 class DeezerCoverSourceResult(CoverSourceResult):
-
     """Deezer search cover result."""
 
     def __init__(self, *args, **kwargs):
@@ -21,7 +20,6 @@ def __init__(self, *args, **kwargs):
 
 
 class DeezerCoverSource(CoverSource):
-
     """
     Cover source using the official Deezer API.
 

sacad/sources/discogs.py

@@ -1,4 +1,4 @@
-""" Discogs cover source. """
+"""Discogs cover source."""
 
 import collections
 import json
@@ -11,7 +11,6 @@
 
 
 class DiscogsCoverSourceResult(CoverSourceResult):
-
     """Discogs search cover result."""
 
     def __init__(self, *args, **kwargs):
@@ -24,7 +23,6 @@ def __init__(self, *args, **kwargs):
 
 
 class DiscogsCoverSource(CoverSource):
-
     """
     Cover source using the official API.
 

sacad/sources/itunes.py

@@ -1,4 +1,4 @@
-""" Itunes cover source. """
+"""Itunes cover source."""
 
 import collections
 import json
@@ -9,7 +9,6 @@
 
 
 class ItunesCoverSourceResult(CoverSourceResult):
-
     """Itunes search cover result."""
 
     def __init__(self, *args, **kwargs):
@@ -21,7 +20,6 @@ def __init__(self, *args, **kwargs):
 
 
 class ItunesCoverSource(CoverSource):
-
     """Itunes cover source."""
 
     SEARCH_URL = "https://itunes.apple.com/search"

sacad/sources/lastfm.py

@@ -1,4 +1,4 @@
-""" LastFM cover source. """
+"""LastFM cover source."""
 
 import collections
 import os.path
@@ -10,7 +10,6 @@
 
 
 class LastFmCoverSourceResult(CoverSourceResult):
-
     """LastFM cover search result."""
 
     def __init__(self, *args, **kwargs):
@@ -22,7 +21,6 @@ def __init__(self, *args, **kwargs):
 
 
 class LastFmCoverSource(CoverSource):
-
     """
     Cover source using the official LastFM API.
 

sacad/tqdm_logging.py

@@ -1,4 +1,4 @@
-""" Code to help using the logging module with tqdm progress bars. """
+"""Code to help using the logging module with tqdm progress bars."""
 
 import contextlib
 import logging
@@ -8,7 +8,6 @@
 
 
 class TqdmLoggingHandler(logging.Handler):
-
     """Logging handler sending messages to the tqdm write method (avoids overlap)."""
 
     def __init__(self, tqdm, *args, **kwargs):

setup.py

@@ -1,6 +1,6 @@
 #!/usr/bin/env python3
 
-""" Package setup. """
+"""Package setup."""
 
 import os
 import re

tests/__init__.py

@@ -1,6 +1,6 @@
 #!/usr/bin/env python3
 
-""" Unit tests for main module. """
+"""Unit tests for main module."""
 
 import asyncio
 import contextlib
@@ -74,7 +74,6 @@ async def delay_coroutine(coroutine, delay):
 
 @unittest.skipUnless(is_internet_reachable(), "Need Internet access")
 class TestSacad(unittest.TestCase):
-
     """Test suite for main module."""
 
     @staticmethod

tests/rate_watcher_test.py

@@ -1,6 +1,6 @@
 #!/usr/bin/env python3
 
-""" Unit tests for rate watcher. """
+"""Unit tests for rate watcher."""
 
 import os
 import tempfile
@@ -15,7 +15,6 @@
 
 
 class TestRateWatcher(unittest.TestCase):
-
     """Test suite for rate watcher."""
 
     def test_minDelayBetweenAccesses(self):

tests/recurse_test.py

@@ -1,6 +1,6 @@
 #!/usr/bin/env python3
 
-""" Unit tests for recurse module. """
+"""Unit tests for recurse module."""
 
 import collections
 import contextlib
@@ -39,7 +39,6 @@ def download(url, filepath):
 
 
 class TestRecursive(unittest.TestCase):
-
     """Test suite for recurse module."""
 
     def __init__(self, *args, **kwargs):