use ruff as formatter

Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>

Author: arthurzam

.github/workflows/test.yml

@@ -108,4 +108,6 @@ jobs:
     steps:
     - name: Checkout code
       uses: actions/checkout@v4
-    - uses: psf/black@stable
+    - uses: astral-sh/ruff-action@v3
+      with:
+        args: "format --check --diff"

Makefile

@@ -15,4 +15,4 @@ clean:
 
 .PHONY: format
 format:
-	$(PYTHON) -m black .
+	$(PYTHON) -m ruff format

pyproject.toml

@@ -89,6 +89,10 @@ exclude = [
 	"doc/api/", "doc/generated/", "doc/_build/",
 ]
 
+[tool.ruff]
+line-length = 100
+target-version = "py311"
+
 [tool.black]
 line-length = 100
 

src/pkgcheck/addons/git.py

@@ -646,7 +646,7 @@ def update_cache(self, force=False):
                 # skip cache usage when not running on the default branch
                 if branch != default_branch:
                     logger.debug(
-                        "skipping %s git repo cache update on " "non-default branch %r",
+                        "skipping %s git repo cache update on non-default branch %r",
                         repo,
                         branch,
                     )

src/pkgcheck/checks/acct.py

@@ -55,7 +55,7 @@ def __init__(self, kind, identifier, **kwargs):
 
     @property
     def desc(self):
-        return f"{self.kind} id {self.identifier} outside permitted " f"static allocation range"
+        return f"{self.kind} id {self.identifier} outside permitted static allocation range"
 
 
 class AcctCheck(GentooRepoCheck, RepoCheck):

src/pkgcheck/checks/codingstyle.py

@@ -356,10 +356,7 @@ def __init__(self, cmd, **kwargs):
 
     @property
     def desc(self):
-        return (
-            f"deprecated insinto usage (use {self.cmd} instead), "
-            f"line {self.lineno}: {self.line}"
-        )
+        return f"deprecated insinto usage (use {self.cmd} instead), line {self.lineno}: {self.line}"
 
 
 class InsintoCheck(Check):

src/pkgcheck/checks/metadata.py

@@ -77,9 +77,7 @@ def __init__(self, license_group, license, restrictions, **kwargs):
     @property
     def desc(self):
         restrictions = " ".join(self.restrictions)
-        return (
-            f"{self.license_group} license {self.license!r} " f'requires RESTRICT="{restrictions}"'
-        )
+        return f'{self.license_group} license {self.license!r} requires RESTRICT="{restrictions}"'
 
 
 class UnnecessaryLicense(results.VersionResult, results.Warning):
@@ -374,8 +372,7 @@ def desc(self):
                 num_profiles = ""
             # collapsed version
             return (
-                f"profile: {self.profile!r}{num_profiles} "
-                f"failed REQUIRED_USE: {self.required_use}"
+                f"profile: {self.profile!r}{num_profiles} failed REQUIRED_USE: {self.required_use}"
             )
         return (
             f"keyword: {self.keyword}, profile: {self.profile!r}, "
@@ -707,7 +704,7 @@ def __init__(self, dep, dep_slots, **kwargs):
 
     @property
     def desc(self):
-        return f"{self.dep!r} matches more than one slot: " f"[ {', '.join(self.dep_slots)} ]"
+        return f"{self.dep!r} matches more than one slot: [ {', '.join(self.dep_slots)} ]"
 
 
 class MissingSlotDepCheck(Check):
@@ -782,8 +779,7 @@ def desc(self):
         s = pluralism(self.pkgs)
         pkgs = ", ".join(self.pkgs)
         return (
-            f'{self.attr}="{self.atom}": USE flag {self.flag!r} missing from '
-            f"package{s}: [ {pkgs} ]"
+            f'{self.attr}="{self.atom}": USE flag {self.flag!r} missing from package{s}: [ {pkgs} ]'
         )
 
 
@@ -1009,8 +1005,7 @@ def __init__(self, attr, atom, age, **kwargs):
     @property
     def desc(self):
         return (
-            f'outdated blocker {self.attr}="{self.atom}": '
-            f"last match removed {self.age} years ago"
+            f'outdated blocker {self.attr}="{self.atom}": last match removed {self.age} years ago'
         )
 
 
@@ -1030,7 +1025,7 @@ def __init__(self, attr, atom, **kwargs):
 
     @property
     def desc(self):
-        return f'nonexistent blocker {self.attr}="{self.atom}": ' "no matches in repo history"
+        return f'nonexistent blocker {self.attr}="{self.atom}": no matches in repo history'
 
 
 class OutdatedBlockersCheck(Check):
@@ -1564,8 +1559,7 @@ def feed(self, pkg):
                             yield BadHomepage(f"HOMEPAGE={homepage!r} lacks protocol", pkg=pkg)
                         elif homepage[:i] not in SrcUriCheck.valid_protos:
                             yield BadHomepage(
-                                f"HOMEPAGE={homepage!r} uses unsupported "
-                                f"protocol {homepage[:i]!r}",
+                                f"HOMEPAGE={homepage!r} uses unsupported protocol {homepage[:i]!r}",
                                 pkg=pkg,
                             )
 

src/pkgcheck/checks/profiles.py

@@ -284,8 +284,7 @@ def _deprecated(self, filename, node, vals):
                 addons.profiles.ProfileNode(pjoin(self.profiles_dir, replacement))
             except profiles_mod.ProfileError:
                 yield ProfileError(
-                    f"nonexistent replacement {replacement!r} "
-                    f"for deprecated profile: {node.name!r}"
+                    f"nonexistent replacement {replacement!r} for deprecated profile: {node.name!r}"
                 )
 
     # non-spec files

src/pkgcheck/checks/python.py

@@ -727,9 +727,7 @@ def __init__(self, filename, uri, **kwargs):
 
     @property
     def desc(self):
-        return (
-            f"GitHub archive {self.filename!r} ({self.uri!r}) is not " "using '.gh.tar.gz' suffix"
-        )
+        return f"GitHub archive {self.filename!r} ({self.uri!r}) is not using '.gh.tar.gz' suffix"
 
 
 class PythonInlinePyPIURI(results.VersionResult, results.Warning):

src/pkgcheck/checks/visibility.py

@@ -12,7 +12,7 @@
 
 
 class FakeConfigurable:
-    "Package wrapper binding profile data." ""
+    "Package wrapper binding profile data."
 
     configurable = True
     __slots__ = ("use", "iuse", "_forced_use", "_masked_use", "_pkg_use", "_raw_pkg", "_profile")
@@ -114,8 +114,7 @@ def desc(self):
             num_profiles = ""
 
         return (
-            f'VCS version visible for KEYWORDS="{self.arch}", '
-            f"profile {self.profile}{num_profiles}"
+            f'VCS version visible for KEYWORDS="{self.arch}", profile {self.profile}{num_profiles}'
         )