.pre-commit-config.yaml

@@ -1,5 +1,5 @@
 default_language_version:
-  python: python3.12
+  python: python3.11
 exclude: ^.*\b(migrations)\b.*$
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
@@ -16,22 +16,45 @@ repos:
       - id: end-of-file-fixer
       - id: trailing-whitespace
   - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: 'v0.3.0'
+    rev: 'v0.1.11'
     hooks:
       - id: ruff
         args:
           - --fix
-      - id: ruff-format
+  - repo: https://github.com/asottile/reorder_python_imports
+    rev: v3.12.0
+    hooks:
+      - id: reorder-python-imports
+        args:
+          - --py310-plus
+          - --application-directories=.:src
+        exclude: migrations/
   - repo: https://github.com/asottile/pyupgrade
-    rev: v3.15.1
+    rev: v3.15.0
     hooks:
       - id: pyupgrade
         args:
           - --py311-plus
         exclude: migrations/
   - repo: https://github.com/adamchainz/django-upgrade
-    rev: 1.16.0
+    rev: 1.15.0
     hooks:
       - id: django-upgrade
         args:
-          - --target-version=5.0
+          - --target-version=4.1
+  - repo: https://github.com/asottile/yesqa
+    rev: v1.5.0
+    hooks:
+      -   id: yesqa
+  - repo: https://github.com/asottile/add-trailing-comma
+    rev: v3.1.0
+    hooks:
+      -   id: add-trailing-comma
+  - repo: https://github.com/hadialqattan/pycln
+    rev: v2.4.0
+    hooks:
+      -   id: pycln
+  - repo: https://github.com/psf/black
+    rev: 23.12.1
+    hooks:
+      - id: black

Dockerfile

@@ -1,4 +1,4 @@
-FROM python:3.12-slim-bullseye
+FROM python:3.11-slim-bullseye
 
 ENV PYTHONFAULTHANDLER=1 \
   PYTHONUNBUFFERED=1 \

pyproject.toml

@@ -104,26 +104,3 @@ follow_imports = "normal"
 [[tool.mypy.overrides]]
 module = "tests.*"
 allow_untyped_defs = true
-
-[tool.ruff]
-target-version = "py312"
-extend-exclude = [
-    ".git",
-    "__pycache__",
-]
-line-length = 120
-
-[tool.ruff.lint]
-select = ["ALL"]
-ignore = [
-    "G004", # Logging statement uses f-string
-    "ANN101", # Missing type annotation for `self` in method
-    "ANN102", # Missing type annotation for `cls` in classmethod
-    "EM101", # Exception must not use a string literal, assign to variable first
-    "EM102", # Exception must not use a f-string literal, assign to variable first
-    "COM812", # missing-trailing-comma (https://docs.astral.sh/ruff/formatter/#conflicting-lint-rules)
-    "ISC001", # single-line-implicit-string-concatenation (https://docs.astral.sh/ruff/formatter/#conflicting-lint-rules)
-]
-
-[tool.ruff.lint.isort]
-force-single-line = true

src/accounting/migrations/0001_initial.py

@@ -7,6 +7,7 @@ from django.db import models
 
 
 class Migration(migrations.Migration):
+
     initial = True
 
     dependencies = [
@@ -98,7 +99,9 @@ class Migration(migrations.Migration):
                 ),
                 (
                     "vat",
-                    djmoney.models.fields.MoneyField(decimal_places=2, max_digits=16, verbose_name="VAT"),
+                    djmoney.models.fields.MoneyField(
+                        decimal_places=2, max_digits=16, verbose_name="VAT"
+                    ),
                 ),
                 ("is_paid", models.BooleanField(default=False, verbose_name="is paid")),
                 (

src/accounting/models.py

@@ -17,7 +17,8 @@ class CreatedModifiedAbstract(models.Model):
 
 
 class Account(CreatedModifiedAbstract):
-    """This is the model where we can give access to several users, such that they
+    """
+    This is the model where we can give access to several users, such that they
     can decide which account to use to pay for something.
     """
 
@@ -29,7 +30,8 @@ class Account(CreatedModifiedAbstract):
 
 
 class Transaction(CreatedModifiedAbstract):
-    """Tracks in and outgoing events of an account. When an order is received, an
+    """
+    Tracks in and outgoing events of an account. When an order is received, an
     amount is subtracted, when a payment is received, an amount is added.
     """
 
@@ -48,7 +50,8 @@ class Transaction(CreatedModifiedAbstract):
 
 
 class Order(CreatedModifiedAbstract):
-    """Scoped out: Contents of invoices will have to be tracked either here or in
+    """
+    Scoped out: Contents of invoices will have to be tracked either here or in
     a separate Invoice model. This is undecided because we are not generating
     invoices at the moment.
     """
@@ -88,7 +91,7 @@ class Order(CreatedModifiedAbstract):
         verbose_name = pgettext_lazy("accounting term", "Order")
         verbose_name_plural = pgettext_lazy("accounting term", "Orders")
 
-    def __str__(self) -> str:
+    def __str__(self):
         return f"Order ID {self.display_id}"
 
 
@@ -113,7 +116,7 @@ class Payment(CreatedModifiedAbstract):
             description=order.description,
         )
 
-    def __str__(self) -> str:
+    def __str__(self):
         return f"Payment ID {self.display_id}"
 
     class Meta:

src/accounting/tests.py

@@ -8,8 +8,8 @@ from . import models
 # do stuff
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
-def test_balance() -> None:
+def test_balance():
     user = User.objects.create_user("test", "lala@adas.com", "1234")
     account = models.Account.objects.create(owner=user)
     assert account.balance == 0

src/membership/__init__.py

@@ -1,4 +1,5 @@
-"""Membership application.
+"""
+Membership application
 ======================
 
 This application's domain relate to organizational structures and

src/membership/apps.py

@@ -5,7 +5,7 @@ from django.db.models.signals import post_migrate
 class MembershipConfig(AppConfig):
     name = "membership"
 
-    def ready(self) -> None:
+    def ready(self):
         from .permissions import persist_permissions
 
         post_migrate.connect(persist_permissions, sender=self)

src/membership/migrations/0001_initial.py

@@ -7,6 +7,7 @@ from django.db import models
 
 
 class Migration(migrations.Migration):
+
     initial = True
 
     dependencies = [

src/membership/migrations/0002_subscriptionperiod_remove_membership_period_and_more.py

@@ -2,11 +2,11 @@
 
 import django.contrib.postgres.constraints
 import django.contrib.postgres.fields.ranges
-from django.db import migrations
+from django.db import migrations, models
-from django.db import models
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
         ("membership", "0001_initial"),
     ]
@@ -34,7 +34,9 @@ class Migration(migrations.Migration):
                 ),
                 (
                     "period",
-                    django.contrib.postgres.fields.ranges.DateRangeField(verbose_name="period"),
+                    django.contrib.postgres.fields.ranges.DateRangeField(
+                        verbose_name="period"
+                    ),
                 ),
             ],
         ),

src/membership/migrations/0003_membership_period.py

@@ -1,11 +1,11 @@
 # Generated by Django 4.1 on 2023-01-02 21:05
 
+from django.db import migrations, models
 import django.db.models.deletion
-from django.db import migrations
-from django.db import models
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
         ("membership", "0002_subscriptionperiod_remove_membership_period_and_more"),
     ]

src/membership/migrations/0004_alter_membership_period.py

@@ -1,11 +1,11 @@
 # Generated by Django 4.1 on 2023-01-02 21:06
 
+from django.db import migrations, models
 import django.db.models.deletion
-from django.db import migrations
-from django.db import models
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
         ("membership", "0003_membership_period"),
     ]

src/membership/migrations/0005_member.py

@@ -4,20 +4,22 @@ from django.db import migrations
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("auth", "0012_alter_user_first_name_max_length"),
+        ('auth', '0012_alter_user_first_name_max_length'),
-        ("membership", "0004_alter_membership_period"),
+        ('membership', '0004_alter_membership_period'),
     ]
 
     operations = [
         migrations.CreateModel(
-            name="Member",
+            name='Member',
-            fields=[],
+            fields=[
+            ],
             options={
-                "proxy": True,
+                'proxy': True,
-                "indexes": [],
+                'indexes': [],
-                "constraints": [],
+                'constraints': [],
             },
-            bases=("auth.user",),
+            bases=('auth.user',),
         ),
     ]

src/membership/models.py

@@ -5,6 +5,7 @@ from django.contrib.postgres.fields import RangeOperators
 from django.db import models
 from django.utils import timezone
 from django.utils.translation import gettext as _
+
 from utils.mixins import CreatedModifiedAbstract
 
 
@@ -34,7 +35,9 @@ class Member(User):
 
 
 class SubscriptionPeriod(CreatedModifiedAbstract):
-    """Denotes a period for which members should pay their membership fee for."""
+    """
+    Denotes a period for which members should pay their membership fee for.
+    """
 
     period = DateRangeField(verbose_name=_("period"))
 
@@ -48,12 +51,16 @@ class SubscriptionPeriod(CreatedModifiedAbstract):
             ),
         ]
 
-    def __str__(self) -> str:
+    def __str__(self):
-        return f"{self.period.lower} - {self.period.upper or _('next general assembly')}"
+        return (
+            f"{self.period.lower} - {self.period.upper or _('next general assembly')}"
+        )
 
 
 class Membership(CreatedModifiedAbstract):
-    """Tracks that a user has membership of a given type for a given period."""
+    """
+    Tracks that a user has membership of a given type for a given period.
+    """
 
     class QuerySet(models.QuerySet):
         def for_member(self, member: Member):
@@ -94,12 +101,13 @@ class Membership(CreatedModifiedAbstract):
         on_delete=models.PROTECT,
     )
 
-    def __str__(self) -> str:
+    def __str__(self):
         return f"{self.user} - {self.period}"
 
 
 class MembershipType(CreatedModifiedAbstract):
-    """Models membership types. Currently only a name, but will in the future
+    """
+    Models membership types. Currently only a name, but will in the future
     possibly contain more information like fees.
     """
 
@@ -109,5 +117,5 @@ class MembershipType(CreatedModifiedAbstract):
 
     name = models.CharField(verbose_name=_("name"), max_length=64)
 
-    def __str__(self) -> str:
+    def __str__(self):
         return self.name

src/membership/permissions.py

@@ -7,7 +7,7 @@ from django.utils.translation import gettext_lazy as _
 PERMISSIONS = []
 
 
-def persist_permissions(sender, **kwargs) -> None:
+def persist_permissions(sender, **kwargs):
     for permission in PERMISSIONS:
         permission.persist_permission()
 
@@ -23,10 +23,10 @@ class Permission:
         PERMISSIONS.append(self)
 
     @property
-    def path(self) -> str:
+    def path(self):
         return f"{self.app_label}.{self.codename}"
 
-    def persist_permission(self) -> None:
+    def persist_permission(self):
         content_type, _ = ContentType.objects.get_or_create(
             app_label=self.app_label,
             model=self.model,

src/membership/views.py

@@ -1,14 +1,15 @@
 from django.utils.translation import gettext_lazy as _
 from django_view_decorator import namespaced_decorator_factory
-from utils.view_utils import RowAction
-from utils.view_utils import render
-from utils.view_utils import render_list
 
 from .permissions import ADMINISTRATE_MEMBERS
 from .selectors import get_member
 from .selectors import get_members
 from .selectors import get_memberships
 from .selectors import get_subscription_periods
+from utils.view_utils import render
+from utils.view_utils import render_list
+from utils.view_utils import RowAction
+
 
 member_view = namespaced_decorator_factory(namespace="member", base_path="membership")
 

src/project/urls.py

@@ -1,4 +1,4 @@
-"""URLs for the membersystem."""
+"""URLs for the membersystem"""
 from django.conf import settings
 from django.contrib import admin
 from django.urls import include
@@ -15,5 +15,4 @@ if settings.DEBUG:
     urlpatterns = [
         path("__debug__/", include("debug_toolbar.urls")),
         path("__reload__/", include("django_browser_reload.urls")),
-        *urlpatterns,
+    ] + urlpatterns
-    ]

src/project/views.py

@@ -1,4 +1,5 @@
 from django_view_decorator import view
+
 from utils.view_utils import render
 
 

src/project/wsgi.py

@@ -1,4 +1,5 @@
-"""WSGI config for membersystem project.
+"""
+WSGI config for membersystem project.
 
 It exposes the WSGI callable as a module-level variable named ``application``.
 

src/utils/templatetags/utils.py

@@ -7,6 +7,7 @@ register = template.Library()
 @register.simple_tag(takes_context=True)
 def active_path(context, path_name, class_name) -> str | None:
     """Return the given class name if the current path matches the given path name."""
+
     path = reverse(path_name)
     request_path = context.get("request").path
 
@@ -18,4 +19,3 @@ def active_path(context, path_name, class_name) -> str | None:
 
     if is_path or is_base_path:
         return class_name
-    return None

src/utils/view_utils.py

@@ -1,24 +1,23 @@
 import contextlib
 from dataclasses import dataclass
-from typing import TYPE_CHECKING
 from typing import Any
 
 from django.contrib.sites.shortcuts import get_current_site
 from django.core.exceptions import FieldError
 from django.core.paginator import Paginator
+from django.db.models import Model
 from django.http import HttpRequest
 from django.http import HttpResponse
 from django.urls import reverse
 from zen_queries import queries_disabled
 from zen_queries import render as zen_queries_render
 
-if TYPE_CHECKING:
-    from django.db.models import Model
-
 
 @dataclass
 class Row:
-    """A row in a table."""
+    """
+    A row in a table.
+    """
 
     data: dict[str, str]
     actions: list[dict[str, str]]
@@ -26,14 +25,18 @@ class Row:
 
 @dataclass
 class RowAction:
-    """An action that can be performed on a row in a table."""
+    """
+    An action that can be performed on a row in a table.
+    """
 
     label: str
     url_name: str
     url_kwargs: dict[str, str]
 
     def render(self, obj) -> dict[str, str]:
-        """Render the action as a dictionary for the given object."""
+        """
+        Render the action as a dictionary for the given object.
+        """
         url = reverse(
             self.url_name,
             kwargs={key: getattr(obj, value) for key, value in self.url_kwargs.items()},
@@ -47,11 +50,14 @@ def render_list(
     entity_name_plural: str,
     objects: list["Model"],
     columns: list[tuple[str, str]],
-    row_actions: list[RowAction] | None = None,
+    row_actions: list[RowAction] = None,
-    list_actions: list[tuple[str, str]] | None = None,
+    list_actions: list[tuple[str, str]] = None,
-    paginate_by: int | None = None,
+    paginate_by: int = None,
 ) -> HttpResponse:
-    """Render a list of objects with a table."""
+    """
+    Render a list of objects with a table.
+    """
+
     # TODO: List actions
 
     total_count = len(objects)
@@ -101,12 +107,16 @@ def render_list(
 
 
 def base_context(request: HttpRequest) -> dict[str, Any]:
-    """Return a base context for all views."""
+    """
+    Return a base context for all views.
+    """
     return {"site": get_current_site(request)}
 
 
 def render(request, template_name, context=None):
-    """Render a template with a base context."""
+    """
+    Render a template with a base context.
+    """
     if context is None:
         context = {}
     context = base_context(request) | context