ruff check; other little fixes

.github/dependabot.yml

@@ -0,0 +1,10 @@
+# Set update schedule for GitHub Actions
+
+version: 2
+updates:
+
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      # Check for updates to GitHub Actions every week
+      interval: "weekly"

README.md

@@ -9,7 +9,7 @@ lab-inventory is licensed for you to use under the BSD 3-Clause License. See COP
 
 ### Quick start
 
-1. Requires Python 3.10+ and Django 4.2+
+1. Requires Python 3.10+. Runs on Django 4.2 LTS and 5.1.
 
 1. Install the package from pypi: `pip install django-lab-inventory`. Worth putting in a virtualenv.
 
@@ -41,4 +41,4 @@ Recommend using [uv](https://docs.astral.sh/uv/) for development.
 
 Run `uv sync` to create a virtual environment and install dependencies. `uv sync --no-dev --frozen` for deployment.
 
-Testing: `uv run pytest`. This uses settings from `test/settings.py`
+Testing: `uv run pytest`. Requires a test database, will use settings from `inventory/test/settings.py`.

inventory/__init__.py

@@ -1,3 +1,3 @@
 # -*- coding: utf-8 -*-
 # -*- mode: python -*-
-__version__ = "0.5.4"
+__version__ = "0.5.5"

inventory/forms.py

@@ -1,10 +1,11 @@
 # -*- coding: utf-8 -*-
 # -*- mode: python -*-
 import datetime
+
 from django import forms
 from django.contrib.auth.models import User
 
-from inventory.models import Account, Order, Vendor, Item, OrderItem
+from inventory.models import Account, Item, Order, OrderItem, Vendor
 
 
 class NewOrderForm(forms.ModelForm):

inventory/migrations/0001_initial.py

@@ -1,10 +1,11 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import models, migrations
-from django.conf import settings
 import datetime
 
+from django.conf import settings
+from django.db import migrations, models
+
 
 class Migration(migrations.Migration):
 

inventory/migrations/0001_squashed_0003_auto_20150626_1807.py

@@ -1,9 +1,10 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import models, migrations
 import datetime
+
 from django.conf import settings
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

inventory/migrations/0002_auto_20150626_1805.py

@@ -1,7 +1,7 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import models, migrations
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

inventory/migrations/0002_auto_20170623_1716.py

@@ -2,9 +2,9 @@
 # Generated by Django 1.11.2 on 2017-06-23 21:16
 from __future__ import unicode_literals
 
+import django.db.models.deletion
 from django.conf import settings
 from django.db import migrations, models
-import django.db.models.deletion
 
 
 class Migration(migrations.Migration):

inventory/migrations/0003_auto_20150626_1807.py

@@ -1,7 +1,7 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import models, migrations
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

inventory/migrations/0007_remove_duplicates.py

@@ -1,6 +1,6 @@
 # Generated by Django 4.1.5 on 2023-07-21 18:17
-from django.db.models import Count
 from django.db import migrations
+from django.db.models import Count
 
 
 def purge_duplicate_named(apps, schema_editor):

inventory/tests/test_models.py

@@ -1,9 +1,10 @@
 # -*- mode: python -*-
 
 import pytest
-
 from django.contrib.auth import get_user_model
-from inventory.models import Item, Order, OrderItem, Vendor, Unit, Category, Account
+
+from inventory.models import Account, Category, Item, Order, OrderItem, Unit, Vendor
+
 
 def get_sentinel_user():
     return get_user_model().objects.get_or_create(username="deleted")[0]

inventory/tests/test_utils.py

@@ -1,7 +1,6 @@
 # -*- mode: python -*-
 
-import pytest
 
 
 def test_dummy():
-    return True
+    pass

inventory/tests/urls.py

@@ -1,4 +1,3 @@
-from django.urls import path
 
 # If you have no specific test URLs, you can use an empty list
 urlpatterns = []

inventory/views.py

@@ -1,19 +1,19 @@
 # -*- coding: utf-8 -*-
 # -*- mode: python -*-
-from django.shortcuts import get_object_or_404, redirect
 import django_filters as filters
 from django.db.models import Q
+from django.shortcuts import get_object_or_404, redirect
 from django.template.response import TemplateResponse
 from django.views import generic
 from django_filters.views import FilterView
 
-from inventory.models import Item, Order, OrderItem
 from inventory.forms import (
-    NewOrderForm,
+    ConfirmOrderForm,
     NewItemForm,
+    NewOrderForm,
     NewOrderItemForm,
-    ConfirmOrderForm,
 )
+from inventory.models import Item, Order, OrderItem
 
 
 def index(request):

pyproject.toml

@@ -52,16 +52,16 @@ include = '\.pyi?$'
 [tool.ruff]
 line-length = 88
 target-version = "py38"
-select = [
+lint.select = [
     "E",  # pycodestyle errors
     "W",  # pycodestyle warnings
     "F",  # pyflakes
     "I",  # isort
     "B",  # flake8-bugbear
 ]
-ignore = ["E221", "E501", "E701"]  # Matching your pep8 ignores
+lint.ignore = ["E221", "E501", "E701"]  # Matching your pep8 ignores
 
-[tool.ruff.per-file-ignores]
+[tool.ruff.lint.per-file-ignores]
 "__init__.py" = ["F401"]
 
 [tool.mypy]

setup.py

@@ -1,4 +1,5 @@
 # -*- coding: utf-8 -*-
 # -*- mode: python -*-
 from setuptools import setup
+
 setup()