Add pre-commit CI

Author: codingjoe

.github/workflows/ci.yml

@@ -1,33 +1,10 @@
 name: CI
-
 on:
   push:
     branches:
       - main
   pull_request:
-
 jobs:
-
-  lint:
-    runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        lint-command:
-          - bandit -r . -x ./tests
-          - black --check --diff .
-          - flake8 .
-          - isort --check-only --diff .
-          - pydocstyle .
-    steps:
-      - uses: actions/checkout@v5
-      - uses: actions/setup-python@v6
-        with:
-          python-version: "3.x"
-          cache: 'pip'
-          cache-dependency-path: pyproject.toml
-      - run: python -m pip install .[lint]
-      - run: ${{ matrix.lint-command }}
-
   docs:
     runs-on: ubuntu-latest
     steps:
@@ -39,7 +16,6 @@ jobs:
       - run: python -m pip install sphinxcontrib-spelling
       - run: python -m pip install -e '.[docs]'
       - run: python -m sphinx -W -b spelling docs docs/_build
-
   dist:
     runs-on: ubuntu-latest
     steps:
@@ -50,11 +26,9 @@ jobs:
       - run: python -m pip install --upgrade pip build wheel twine readme-renderer
       - run: python -m build --sdist --wheel
       - run: python -m twine check dist/*
-
   PyTest:
     needs:
       - dist
-      - lint
     runs-on: ubuntu-latest
     strategy:
       matrix:

.github/workflows/release.yml

@@ -1,11 +1,8 @@
 name: Release
-
 on:
   release:
     types: [published]
-
 jobs:
-
   PyPi:
     runs-on: ubuntu-latest
     steps:

.pre-commit-config.yaml

@@ -0,0 +1,53 @@
+# See https://pre-commit.com for more information
+# See https://pre-commit.com/hooks.html for more hooks
+repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v6.0.0
+    hooks:
+      - id: trailing-whitespace
+      - id: check-merge-conflict
+      - id: check-ast
+      - id: check-toml
+      - id: check-yaml
+      - id: debug-statements
+      - id: end-of-file-fixer
+      - id: name-tests-test
+        args: ['--pytest-test-first']
+        exclude: ^tests/(testapp/|manage.py$)
+      - id: no-commit-to-branch
+        args: [--branch, main]
+  - repo: https://github.com/asottile/pyupgrade
+    rev: v3.21.1
+    hooks:
+      - id: pyupgrade
+  - repo: https://github.com/adamchainz/django-upgrade
+    rev: 1.29.1
+    hooks:
+      - id: django-upgrade
+  - repo: https://github.com/hukkin/mdformat
+    rev: 1.0.0
+    hooks:
+      - id: mdformat
+        additional_dependencies:
+          - mdformat-ruff
+          - mdformat-footnote
+          - mdformat-gfm
+          - mdformat-gfm-alerts
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.14.4
+    hooks:
+      - id: ruff-check
+        args: [--fix, --exit-non-zero-on-fix]
+      - id: ruff-format
+  - repo: https://github.com/google/yamlfmt
+    rev: v0.20.0
+    hooks:
+      - id: yamlfmt
+  - repo: https://github.com/djlint/djLint
+    rev: v1.36.4
+    hooks:
+      - id: djlint-reformat-django
+ci:
+  autoupdate_schedule: weekly
+  skip:
+    - no-commit-to-branch

.readthedocs.yaml

@@ -1,19 +1,15 @@
 # .readthedocs.yaml
 # Read the Docs configuration file
 # See https://docs.readthedocs.io/en/stable/config-file/v2.html for details
-
 version: 2
-
 build:
   os: ubuntu-20.04
   tools:
     python: "3.11"
   apt_packages:
     - graphviz
-
 sphinx:
   configuration: docs/conf.py
-
 python:
   install:
     - method: pip

CONTRIBUTING.md

@@ -27,7 +27,6 @@ to add code to their runtime environment that they don't need.
 All features need to be tested. A CI suite should be in place. Running and
 writing tests should be reasonably accessible for first time contributors.
 
-
 ## Release
 
 We follow [semantic versioning](https://semver.org/). To release a new version

docs/conf.py

@@ -49,7 +49,7 @@ def linkcode_resolve(domain, info):
             pass
         try:
             lines, first_line = inspect.getsourcelines(item)
-            lineno = "#L%d-L%s" % (first_line, first_line + len(lines) - 1)
+            lineno = f"#L{first_line:d}-L{first_line + len(lines) - 1:d}"
         except (TypeError, OSError):
             pass
     return (
@@ -79,6 +79,9 @@ def linkcode_resolve(domain, info):
 
 graphviz_output_format = "svg"
 
-inheritance_graph_attrs = dict(
-    rankdir="TB", size='"6.0, 8.0"', fontsize=14, ratio="compress"
-)
+inheritance_graph_attrs = {
+    "rankdir": "TB",
+    "size": '"6.0, 8.0"',
+    "fontsize": 14,
+    "ratio": "compress",
+}

docs/tutorial/testing.rst

@@ -90,4 +90,3 @@ running workflow. You can test any other task by simply creating the
 workflow and task in during test setup. In those cases you will need
 pass the task primary key. You can find more information about this
 in the :ref:`URLs documentation<topic-urls>`.
-

joeflow/admin.py

@@ -1,7 +1,7 @@
 from django.contrib import admin, messages
 from django.contrib.auth import get_permission_codename
 from django.db import transaction
-from django.forms.widgets import MediaAsset, Media, Script
+from django.forms.widgets import Media, MediaAsset, Script
 from django.utils.html import format_html
 from django.utils.safestring import mark_safe
 from django.utils.translation import gettext_lazy as t
@@ -21,13 +21,13 @@ def rerun(modeladmin, request, queryset):
     if succeeded:
         messages.warning(
             request,
-            "Only failed tasks can be retried. %s tasks have been skipped" % succeeded,
+            f"Only failed tasks can be retried. {succeeded} tasks have been skipped",
         )
     counter = 0
     for obj in queryset.not_succeeded().iterator():
         obj.enqueue()
         counter += 1
-    messages.success(request, "%s tasks have been successfully queued" % counter)
+    messages.success(request, f"{counter} tasks have been successfully queued")
 
 
 @admin.action(
@@ -39,8 +39,7 @@ def cancel(modeladmin, request, queryset):
     if not_scheduled:
         messages.warning(
             request,
-            "Only scheduled tasks can be canceled. %s tasks have been skipped"
-            % not_scheduled,
+            f"Only scheduled tasks can be canceled. {not_scheduled} tasks have been skipped",
         )
     queryset.scheduled().cancel(request.user)
     messages.success(request, "Tasks have been successfully canceled")
@@ -138,12 +137,11 @@ class TaskInlineAdmin(admin.TabularInline):
 
 
 class CSS(MediaAsset):
-
     element_template = "<style{attributes}>{path}</style>"
 
     @property
     def path(self):
-        return mark_safe(self._path)
+        return mark_safe(self._path)  # noqa: S308
 
 
 class WorkflowAdmin(VersionAdmin):
@@ -168,7 +166,7 @@ def get_readonly_fields(self, *args, **kwargs):
     def display_workflow_diagram(self, obj):
         """Display workflow diagram using MermaidJS for client-side rendering."""
         if obj.pk:
-            return mark_safe(
+            return mark_safe(  # noqa: S308
                 f"""<pre class="mermaid" style="width: 100%; display: block">{obj.get_instance_graph_mermaid()}</pre>"""
             )
         return ""

joeflow/conf.py

@@ -5,8 +5,7 @@
 
 
 class JoeflowAppConfig(AppConf):
-    """
-    List of available settings.
+    """List of available settings.
 
     To change the default values just set the setting in your settings file.
     """

joeflow/management/commands/render_workflow_graph.py

@@ -53,8 +53,7 @@ def handle(self, *args, **options):
                 opt = workflow._meta
                 if verbosity > 0:
                     self.stdout.write(
-                        "Rendering graph for '%s.%s'… "
-                        % (opt.app_label, opt.model_name),
+                        f"Rendering graph for '{opt.app_label}.{opt.model_name}'… ",
                         ending="",
                     )
                 filename = f"{opt.app_label}_{workflow.__name__}".lower()
@@ -65,5 +64,5 @@ def handle(self, *args, **options):
                     self.stdout.write("Done!", self.style.SUCCESS)
             else:
                 self.stderr.write(
-                    "%r is not a Workflow subclass" % workflow, self.style.WARNING
+                    f"{workflow!r} is not a Workflow subclass", self.style.WARNING
                 )