From 8b09f4feb5395c4a067ca9085d5d1f0f2ea05087 Mon Sep 17 00:00:00 2001 From: Daniel Bauer Date: Mon, 11 Nov 2024 09:40:52 +0100 Subject: [PATCH] linting --- .flake8 | 3 +++ django_signposting/middleware.py | 1 - django_signposting/utils.py | 4 ++-- example/example/urls.py | 1 - example/example/views.py | 5 +++-- pyproject.toml | 2 +- tests/conftest.py | 1 - tests/test_middleware.py | 4 ++-- tests/test_utils.py | 5 ++--- 9 files changed, 13 insertions(+), 13 deletions(-) create mode 100644 .flake8 diff --git a/.flake8 b/.flake8 new file mode 100644 index 0000000..20ab825 --- /dev/null +++ b/.flake8 @@ -0,0 +1,3 @@ +[flake8] +max-line-length = 99 +exclude = .git, __pycache__, .pytest_cache, .venv, venv, build \ No newline at end of file diff --git a/django_signposting/middleware.py b/django_signposting/middleware.py index 41b6f48..dcf8d54 100644 --- a/django_signposting/middleware.py +++ b/django_signposting/middleware.py @@ -36,4 +36,3 @@ class SignpostingMiddleware: link_snippets[-1] += f' ; type="{signpost.type}"' response["Link"] = " , ".join(link_snippets) - diff --git a/django_signposting/utils.py b/django_signposting/utils.py index 955728d..f86c8bf 100644 --- a/django_signposting/utils.py +++ b/django_signposting/utils.py @@ -13,5 +13,5 @@ def add_signposts(response: HttpResponse, *args: Signpost): response._signposts = [] for signpost in args: - if not signpost in response._signposts: - response._signposts.append(signpost) \ No newline at end of file + if signpost not in response._signposts: + response._signposts.append(signpost) diff --git a/example/example/urls.py b/example/example/urls.py index 7684e63..de9b45b 100644 --- a/example/example/urls.py +++ b/example/example/urls.py @@ -14,7 +14,6 @@ Including another URLconf 1. Import the include() function: from django.urls import include, path 2. Add a URL to urlpatterns: path('blog/', include('blog.urls')) """ -from django.contrib import admin from django.urls import path from . import views diff --git a/example/example/views.py b/example/example/views.py index 7f01760..676b372 100644 --- a/example/example/views.py +++ b/example/example/views.py @@ -3,9 +3,10 @@ from django_signposting.utils import add_signposts from signposting import Signpost, LinkRel + def my_view(request): response = HttpResponse("Hello, world!") - + # Add signpostings as string add_signposts( response, @@ -13,4 +14,4 @@ def my_view(request): Signpost(LinkRel.author, "https://orcid.org/0000-0001-9447-460X") ) - return response \ No newline at end of file + return response diff --git a/pyproject.toml b/pyproject.toml index 17e8fd2..a98eb8f 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -28,4 +28,4 @@ dev = [ [pytest] minversion = 6.0 addopts = "-ra -q" -testpaths = ["tests"] +testpaths = ["tests"] \ No newline at end of file diff --git a/tests/conftest.py b/tests/conftest.py index cf1af3d..6695a93 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -1,4 +1,3 @@ -import pytest from django.conf import settings diff --git a/tests/test_middleware.py b/tests/test_middleware.py index 0367ced..ed065eb 100644 --- a/tests/test_middleware.py +++ b/tests/test_middleware.py @@ -1,8 +1,8 @@ -from re import A from django.http import HttpResponse from django_signposting.middleware import SignpostingMiddleware from signposting import LinkRel, Signpost + def test_middleware_no_signposting(): response = HttpResponse() response.status_code = 200 @@ -74,4 +74,4 @@ def test_middleware_type_link(): middleware = SignpostingMiddleware(lambda request: response) response = middleware(None) - assert response.headers["Link"] == ' ; rel="type"' \ No newline at end of file + assert response.headers["Link"] == ' ; rel="type"' diff --git a/tests/test_utils.py b/tests/test_utils.py index da254fd..bb33e48 100644 --- a/tests/test_utils.py +++ b/tests/test_utils.py @@ -5,7 +5,7 @@ from signposting import Signpost, LinkRel def test_add_signpost(): response = HttpResponse() - add_signposts(response, Signpost(LinkRel.item,"http://example.com")) + add_signposts(response, Signpost(LinkRel.item, "http://example.com")) assert len(response._signposts) == 1 @@ -18,7 +18,6 @@ def test_add_multiple_signposts(): Signpost(LinkRel.author, "https://example3.com"), ) - assert len(response._signposts) == 3 @@ -35,4 +34,4 @@ def test_add_signpost_duplicate(): add_signposts(response, Signpost(LinkRel.item, "http://example.com")) add_signposts(response, Signpost(LinkRel.item, "http://example.com")) - assert len(response._signposts) == 1 \ No newline at end of file + assert len(response._signposts) == 1