Toughen rough linting (#385)

# Changes

## ruff linting

### flake8-commas (COM)

- https://docs.astral.sh/ruff/rules/#flake8-commas-com
- https://pypi.org/project/flake8-commas/

### flake8-builtins (A)

- https://docs.astral.sh/ruff/rules/#flake8-builtins-a
- https://pypi.org/project/flake8-builtins/

### flake8-errmsg (EM)

- https://docs.astral.sh/ruff/rules/#flake8-errmsg-em
- https://pypi.org/project/flake8-errmsg/

CHANGES

@@ -4,6 +4,25 @@
 
 - _Add your latest changes from PRs here_
 
+### Development
+
+- Strengthen linting (#385)
+
+  - Add flake8-commas (COM)
+
+    - https://docs.astral.sh/ruff/rules/#flake8-commas-com
+    - https://pypi.org/project/flake8-commas/
+
+  - Add flake8-builtins (A)
+
+    - https://docs.astral.sh/ruff/rules/#flake8-builtins-a
+    - https://pypi.org/project/flake8-builtins/
+
+  - Add flake8-errmsg (EM)
+
+    - https://docs.astral.sh/ruff/rules/#flake8-errmsg-em
+    - https://pypi.org/project/flake8-errmsg/
+
 ## django-slugify-processor 1.6.0 (2023-12-09)
 
 _Maintenance only, no bug fixes or features_

docs/conf.py

@@ -51,7 +51,7 @@
 master_doc = "index"
 
 project = about["__title__"]
-copyright = about["__copyright__"]
+project_copyright = about["__copyright__"]
 
 version = "%s" % (".".join(about["__version__"].split("."))[:2])
 release = "%s" % (about["__version__"])
@@ -94,7 +94,7 @@
         "sidebar/navigation.html",
         "sidebar/projects.html",
         "sidebar/scroll-end.html",
-    ]
+    ],
 }
 
 # linkify_issues

pyproject.toml

@@ -136,8 +136,11 @@ select = [
   "F", # pyflakes
   "I", # isort
   "UP", # pyupgrade
+  "A", # flake8-builtins
   "B", # flake8-bugbear
   "C4", # flake8-comprehensions
+  "COM", # flake8-commas
+  "EM", # flake8-errmsg
   "Q", # flake8-quotes
   "PTH", # flake8-use-pathlib
   "SIM", # flake8-simplify
@@ -146,6 +149,9 @@ select = [
   "RUF", # Ruff-specific rules
   "D", # pydocstyle
 ]
+ignore = [
+  "COM812", # missing trailing comma, ruff format conflict
+]
 
 [tool.ruff.lint.isort]
 known-first-party = [

test_app/models.py

@@ -10,5 +10,6 @@ class MyModel(models.Model):
 
     title = models.CharField(max_length=255)
     django_extensions_slug = django_extensions.db.fields.AutoSlugField(
-        populate_from="title", slugify_function=slugify
+        populate_from="title",
+        slugify_function=slugify,
     )

tests/settings.py

@@ -5,7 +5,7 @@
 SECRET_KEY: str = os.getenv("SECRET_KEY", "dummy")
 
 DATABASES: t.Dict[str, t.Any] = {
-    "default": {"NAME": ":memory:", "ENGINE": "django.db.backends.sqlite3"}
+    "default": {"NAME": ":memory:", "ENGINE": "django.db.backends.sqlite3"},
 }
 
 INSTALLED_APPS: t.List[str] = ["test_app"]

tests/test_filters.py

@@ -12,9 +12,9 @@ def test_slugify_via_builtin_override(settings: t.Any) -> None:
             "BACKEND": "django.template.backends.django.DjangoTemplates",
             "APP_DIRS": True,
             "OPTIONS": {
-                "builtins": ["django_slugify_processor.templatetags.slugify_processor"]
+                "builtins": ["django_slugify_processor.templatetags.slugify_processor"],
             },
-        }
+        },
     ]
 
     template = Template("{{'c++'|slugify}}")
@@ -26,7 +26,10 @@ def test_slugify_via_load_templatetags(settings: t.Any) -> None:
     settings.SLUGIFY_PROCESSORS = ["test_app.coding.slugify_programming"]
     settings.INSTALLED_APPS = ["django_slugify_processor"]
     settings.TEMPLATES = [
-        {"BACKEND": "django.template.backends.django.DjangoTemplates", "APP_DIRS": True}
+        {
+            "BACKEND": "django.template.backends.django.DjangoTemplates",
+            "APP_DIRS": True,
+        },
     ]
 
     template = Template('{% load slugify_processor %}{{"c++"|slugify}}')