ci(ruff): enable ruff-format (#2599)

ci(ruff): enable ruff-format

Enable ruff-format
Disable black

Fixes #2597

Reviewed-by: Laura Barcziová
Reviewed-by: Nikola Forró

Author: softwarefactory-project-zuul[bot]

.pre-commit-config.yaml

@@ -3,34 +3,12 @@
 # pre-commit install -t pre-commit -t pre-push
 
 repos:
-  - repo: https://github.com/psf/black
-    rev: 24.8.0
-    hooks:
-      - id: black
-  - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: v4.0.0-alpha.8
-    hooks:
-      - id: prettier
-        exclude: tests_openshift/openshift_integration/test_data/
-  - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.6.0
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.7.0
     hooks:
-      - id: check-added-large-files
-      - id: check-ast
-      - id: check-builtin-literals
-      - id: check-docstring-first
-      - id: check-executables-have-shebangs
-      - id: check-merge-conflict
-      - id: check-symlinks
-      - id: check-yaml
-        args: [--allow-multiple-documents]
-      - id: detect-private-key
-        exclude: tests/conftest.py
-      - id: detect-aws-credentials
-        args: [--allow-missing-credentials]
-      - id: end-of-file-fixer
-      - id: mixed-line-ending
-      - id: trailing-whitespace
+      - id: ruff
+        args: [--fix, --exit-non-zero-on-fix]
+      - id: ruff-format
   - repo: https://github.com/pre-commit/mirrors-mypy
     rev: v1.11.2
     hooks:
@@ -57,6 +35,30 @@ repos:
             types-cachetools,
           ]
         exclude: files/scripts/
+  - repo: https://github.com/pre-commit/mirrors-prettier
+    rev: v4.0.0-alpha.8
+    hooks:
+      - id: prettier
+        exclude: tests_openshift/openshift_integration/test_data/
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v4.6.0
+    hooks:
+      - id: check-added-large-files
+      - id: check-ast
+      - id: check-builtin-literals
+      - id: check-docstring-first
+      - id: check-executables-have-shebangs
+      - id: check-merge-conflict
+      - id: check-symlinks
+      - id: check-yaml
+        args: [--allow-multiple-documents]
+      - id: detect-private-key
+        exclude: tests/conftest.py
+      - id: detect-aws-credentials
+        args: [--allow-missing-credentials]
+      - id: end-of-file-fixer
+      - id: mixed-line-ending
+      - id: trailing-whitespace
   - repo: https://github.com/shellcheck-py/shellcheck-py
     rev: v0.10.0.1
     hooks:
@@ -116,9 +118,3 @@ repos:
           - LICENSE_HEADER.txt
           - --comment-style
           - "#"
-  - repo: https://github.com/astral-sh/ruff-pre-commit
-    # Ruff version.
-    rev: v0.0.291
-    hooks:
-      - id: ruff
-        args: [--fix, --exit-non-zero-on-fix]

alembic/versions/0ad4d1c2a2d8_srpmbuildmodel_copr_related_fields.py

@@ -29,7 +29,7 @@
 from alembic import op
 from packit_service.models import ProjectAndEventsConnector
 
-if TYPE_CHECKING:  # noqa: SIM108
+if TYPE_CHECKING:
     Base = object
 else:
     Base = declarative_base()

alembic/versions/46b37040cb19_add_koji_build_scratch_attribute.py

@@ -24,7 +24,7 @@
 depends_on = None
 
 
-if TYPE_CHECKING:  # noqa: SIM108
+if TYPE_CHECKING:
     Base = object
 else:
     Base = declarative_base()

alembic/versions/68ce6a306b30_add_koji_build_groups.py

@@ -32,7 +32,7 @@
 from alembic import op
 from packit_service.models import ProjectAndEventsConnector
 
-if TYPE_CHECKING:  # noqa: SIM108
+if TYPE_CHECKING:
     Base = object
 else:
     Base = declarative_base()

alembic/versions/70c369f7ba80_add_explicit_mapping_of_tf_to_copr.py

@@ -30,7 +30,7 @@
 from alembic import op
 from packit_service.models import ProjectAndEventsConnector
 
-if TYPE_CHECKING:  # noqa: SIM108
+if TYPE_CHECKING:
     Base = object
 else:
     Base = declarative_base()

alembic/versions/726cb0f70c6d_link_srpmbuildmodel_to_jobtriggermodel.py

@@ -36,7 +36,7 @@
 depends_on = None
 
 
-if TYPE_CHECKING:  # noqa: SIM108
+if TYPE_CHECKING:
     Base = object
 else:
     Base = declarative_base()

alembic/versions/7d2b9fb924fa_add_sync_release_pull_request_table.py

@@ -20,7 +20,7 @@
 
 from alembic import op
 
-if TYPE_CHECKING:  # noqa: SIM108
+if TYPE_CHECKING:
     Base = object
 else:
     Base = declarative_base()
@@ -181,7 +181,9 @@ def upgrade():
                 pr_id = int(url[(url.rfind("/pull-request/") + 14) :])  # noqa[203]
                 namespace = "rpms"
                 repo = url[
-                    (url.rfind("rpms/") + 5) : url.rfind("/pull-request")  # noqa[203]
+                    (url.rfind("rpms/") + 5) : url.rfind(
+                        "/pull-request",
+                    )  # noqa[203]
                 ]
                 pull_request = SyncReleasePullRequestModel.get_or_create(
                     pr_id,

alembic/versions/800abbbb23c9_allow_multiple_forges_in_allowlist.py

@@ -25,7 +25,7 @@
 
 
 # https://github.com/python/mypy/issues/2477#issuecomment-313984522 ^_^
-if TYPE_CHECKING:  # noqa: SIM108
+if TYPE_CHECKING:
     Base = object
 else:
     Base = declarative_base()

alembic/versions/8fee25b27402_drop_merge_duplicate_gitlab_projects.py

@@ -15,7 +15,7 @@
 from alembic import op
 
 # https://github.com/python/mypy/issues/2477#issuecomment-313984522 ^_^
-if TYPE_CHECKING:  # noqa: SIM108
+if TYPE_CHECKING:
     Base = object
 else:
     Base = declarative_base()
@@ -72,9 +72,7 @@ def upgrade():
             len(project.project_authentication_issue) <= 0
             and len(project_dot_git.project_authentication_issue) > 0
         ):
-            project_authentication_issue = project_dot_git.project_authentication_issue[
-                0
-            ]
+            project_authentication_issue = project_dot_git.project_authentication_issue[0]
             project_authentication_issue.project_id = project.id
             session.add(project_authentication_issue)
             # we need to commit here explicitly b/c we are changing the foreign key

alembic/versions/9fe0e970880f_add_copr_build_groups.py

@@ -32,7 +32,7 @@
 from alembic import op
 from packit_service.models import ProjectAndEventsConnector
 
-if TYPE_CHECKING:  # noqa: SIM108
+if TYPE_CHECKING:
     Base = object
 else:
     Base = declarative_base()