chore: enforce --name/-n argument in makemigrations management command

api/core/management/commands/makemigrations.py

@@ -0,0 +1,18 @@
+from django.core.management import CommandError
+from django.core.management.commands.makemigrations import (
+    Command as MakeMigrationsCommand,
+)
+
+
+class Command(MakeMigrationsCommand):
+    """
+    Customise the makemigrations command to enforce use of `--name/-n` argument.
+    """
+
+    def handle(self, *app_labels, **options):
+        if not options.get("name") and not (
+            options.get("check_changes") or options.get("dry_run")
+        ):
+            raise CommandError("--name/-n is a required argument")
+
+        return super().handle(*app_labels, **options)

api/tests/unit/core/management/test_unit_core_management_makemigrations.py

@@ -0,0 +1,22 @@
+import pytest
+from django.core.management import CommandError, call_command
+
+
+def test_makemigrations_without_name_raises_error():
+    with pytest.raises(CommandError, match="--name/-n is a required argument"):
+        call_command('makemigrations')
+
+
+@pytest.mark.django_db(databases=["default", "analytics"])
+def test_makemigrations_with_check_changes_runs_without_error():
+    call_command('makemigrations', check_changes=True)
+
+
+@pytest.mark.django_db(databases=["default", "analytics"])
+def test_makemigrations_with_dry_run_runs_without_error():
+    call_command('makemigrations', dry_run=True)
+
+
+@pytest.mark.django_db(databases=["default", "analytics"])
+def test_makemigrations_with_name_runs_without_error():
+    call_command('makemigrations', name="some_useful_name")