Switch to TOML file and ruff.

Author: tim-schilling

.github/workflows/test.yml

@@ -33,12 +33,12 @@ jobs:
         with:
           python-version: ${{ matrix.python-version }}
           cache: pip
-          cache-dependency-path: 'requirements.txt'
+          cache-dependency-path: 'pyproject.toml'
 
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip setuptools wheel
-          python -m pip install -r requirements.txt
+          python -m pip install .
 
       - name: Run tests
         run: |

.pre-commit-config.yaml

@@ -1,43 +1,30 @@
 repos:
 -   repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.3.0
+    rev: v5.0.0
     hooks:
     -   id: check-yaml
     -   id: end-of-file-fixer
     -   id: trailing-whitespace
     -   id: mixed-line-ending
--   repo: https://github.com/pycqa/flake8
-    rev: 4.0.1
+-   repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.9.4
     hooks:
-    -   id: flake8
+    -   id: ruff
+        args: [--fix]
+    -   id: ruff-format
 -   repo: https://github.com/pycqa/doc8
-    rev: 0.11.2
+    rev: v1.1.2
     hooks:
     -   id: doc8
--   repo: https://github.com/asottile/pyupgrade
-    rev: v2.34.0
-    hooks:
-    -   id: pyupgrade
-        args: [--py38-plus]
 -   repo: https://github.com/adamchainz/django-upgrade
-    rev: 1.7.0
+    rev: 1.22.2
     hooks:
     -   id: django-upgrade
-        args: [--target-version, "3.2"]
--   repo: https://github.com/pycqa/isort
-    rev: 5.10.1
-    hooks:
-    -   id: isort
+        args: [--target-version, "5.0"]
 -   repo: https://github.com/pre-commit/pygrep-hooks
-    rev: v1.9.0
+    rev: v1.10.0
     hooks:
     -   id: python-check-blanket-noqa
     -   id: python-check-mock-methods
     -   id: python-no-eval
     -   id: python-no-log-warn
--   repo: https://github.com/psf/black
-    rev: 22.3.0
-    hooks:
-    -   id: black
-        language_version: python3
-        entry: black --target-version=py38

README.md

@@ -78,7 +78,7 @@ warning. Do what's comfortable.
 
 If not, I highly recommend using the Windows Subsystem for Linux
 ([docs](https://learn.microsoft.com/en-us/windows/wsl/about)). If you do, the
-rest of the instructions will work for you. If you don't have access to that, 
+rest of the instructions will work for you. If you don't have access to that,
 please scroll down to [Windows non-WSL Setup](#windows-non-wsl-setup).
 
 ---
@@ -95,7 +95,7 @@ please scroll down to [Windows non-WSL Setup](#windows-non-wsl-setup).
    ```
 3. Install the project dependencies.
    ```shell
-   pip install -r requirements.txt
+   pip install -e .
    ```
 4. Create a new .env file.
    ```shell
@@ -151,7 +151,7 @@ Proceed to [Lab 1](docs/lab1.md).
    ```
 3. Install the project dependencies.
    ```shell
-   pip install -r requirements.txt
+   pip install -e .
    ```
 4. Create a new .env file.
    ```shell

docs/lab1.md

@@ -168,7 +168,7 @@ If you've gone through all the steps of the investigation, and you're still unsu
 file isn't being shared, you should read about the
 [``enctype`` attribute of forms](https://developer.mozilla.org/en-US/docs/Web/HTML/Reference/Elements/form#enctype).
 
-You'll see that the 
+You'll see that the
 
 By default, a ``<form>`` element will use ``enctype="application/x-www-form-urlencoded"``.
 However, as we found, that will not submit any ``<input type="file" />`` (or other file-based

docs/lab2.md

@@ -36,7 +36,7 @@ cache.clear()
 Let's consider what we know:
 
 - The pages render correctly and there are no errors.
-- The pages rendered "fast" earlier, but over time as the data set has grown, 
+- The pages rendered "fast" earlier, but over time as the data set has grown,
   they have slowed down.
 
 
@@ -449,7 +449,7 @@ At this point it's clear there's a difference in functionality between how
 ``toggle_post_privacy`` and ``update_post`` save the data changes.
 ``toggle_post_privacy`` updates the instance using a ``QuerySet.update()``
 call while ``update_post`` uses ``PostForm.save()`` which under the hood is
-doing something like ``Model.save()``. If we read 
+doing something like ``Model.save()``. If we read
 [the documentation for ``update``](https://docs.djangoproject.com/en/stable/ref/models/querysets/#update)
 we'll see that there's a note that the ``post_save`` signal isn't emitted.
 This explains why ``toggle_post_privacy`` doesn't delete the cached value

manage.py

@@ -1,5 +1,6 @@
 #!/usr/bin/env python
 """Django's command-line utility for administrative tasks."""
+
 import os
 import sys
 

project/data/management/commands/fake_data.py

@@ -21,7 +21,7 @@ def log(event):
     logger.info(f"{event} - start")
     yield
     end = timezone.now()
-    logger.info(f"{event} - end, {(end-start).total_seconds()}s")
+    logger.info(f"{event} - end, {(end - start).total_seconds()}s")
 
 
 class Command(BaseCommand):

project/data/markdown.py

@@ -1,4 +1,4 @@
-from datetime import datetime, timezone
+from datetime import UTC, datetime
 from itertools import cycle
 
 from faker import Faker
@@ -42,17 +42,17 @@ def generate_data(user, image_category, post_categories):
         slug = slugify(title) + f"-{fake.pyint(10, 99999)}"
         publish_at = (
             fake.date_time_between_dates(
-                datetime_start=datetime(2020, 1, 1, tzinfo=timezone.utc),
-                datetime_end=datetime(2022, 10, 12, tzinfo=timezone.utc),
-                tzinfo=timezone.utc,
+                datetime_start=datetime(2020, 1, 1, tzinfo=UTC),
+                datetime_end=datetime(2022, 10, 12, tzinfo=UTC),
+                tzinfo=UTC,
             )
             if fake.pybool()
             else None
         )
         created = fake.date_time_between_dates(
-            datetime_start=datetime(2020, 1, 1, tzinfo=timezone.utc),
-            datetime_end=datetime(2022, 10, 12, tzinfo=timezone.utc),
-            tzinfo=timezone.utc,
+            datetime_start=datetime(2020, 1, 1, tzinfo=UTC),
+            datetime_end=datetime(2022, 10, 12, tzinfo=UTC),
+            tzinfo=UTC,
         )
         if publish_at and publish_at < created:
             created = publish_at
@@ -89,17 +89,17 @@ def generate_data(user, image_category, post_categories):
         slug = slugify(title) + f"-{fake.pyint(10, 9999)}"
         publish_at = (
             fake.date_time_between_dates(
-                datetime_start=datetime(2020, 1, 1, tzinfo=timezone.utc),
-                datetime_end=datetime(2022, 10, 12, tzinfo=timezone.utc),
-                tzinfo=timezone.utc,
+                datetime_start=datetime(2020, 1, 1, tzinfo=UTC),
+                datetime_end=datetime(2022, 10, 12, tzinfo=UTC),
+                tzinfo=UTC,
             )
             if fake.pybool()
             else None
         )
         created = fake.date_time_between_dates(
-            datetime_start=datetime(2020, 1, 1, tzinfo=timezone.utc),
-            datetime_end=datetime(2022, 10, 12, tzinfo=timezone.utc),
-            tzinfo=timezone.utc,
+            datetime_start=datetime(2020, 1, 1, tzinfo=UTC),
+            datetime_end=datetime(2022, 10, 12, tzinfo=UTC),
+            tzinfo=UTC,
         )
         if publish_at and publish_at < created:
             created = publish_at

project/data/subscribers.py

@@ -1,5 +1,5 @@
 from collections import OrderedDict
-from datetime import datetime, timezone
+from datetime import UTC, datetime
 from functools import partial
 
 from django.contrib.auth.models import User
@@ -25,9 +25,9 @@ def generate_data(categories: CategoryData):
             )
             user.email = f"{user.username}@example.com"
             user.date_joined = fake.date_time_between_dates(
-                datetime_start=datetime(2020, 1, 1, tzinfo=timezone.utc),
-                datetime_end=datetime(2022, 10, 12, tzinfo=timezone.utc),
-                tzinfo=timezone.utc,
+                datetime_start=datetime(2020, 1, 1, tzinfo=UTC),
+                datetime_end=datetime(2022, 10, 12, tzinfo=UTC),
+                tzinfo=UTC,
             )
             users.append(user)
         User.objects.bulk_create(users, ignore_conflicts=True)
@@ -50,9 +50,9 @@ def generate_data(categories: CategoryData):
     for i in range(0, USER_COUNT, 50):
         created_map = {
             user_id: fake.date_time_between_dates(
-                datetime_start=datetime(2020, 1, 1, tzinfo=timezone.utc),
-                datetime_end=datetime(2022, 10, 12, tzinfo=timezone.utc),
-                tzinfo=timezone.utc,
+                datetime_start=datetime(2020, 1, 1, tzinfo=UTC),
+                datetime_end=datetime(2022, 10, 12, tzinfo=UTC),
+                tzinfo=UTC,
             )
             for user_id in user_ids[i : i + 50]
         }

project/newsletter/migrations/0001_initial.py

@@ -7,7 +7,6 @@
 
 
 class Migration(migrations.Migration):
-
     initial = True
 
     dependencies = [