just skip first param instead of by name "request"

this re-enables (worked before 1.0) view signatures like

`def view(_: HttpRequest)` to avoid linters complaining about the unused
`request` param

ninja/signature/details.py

@@ -1,6 +1,7 @@
 import inspect
 import warnings
 from collections import defaultdict, namedtuple
+from itertools import islice
 from typing import Any, Callable, Dict, Generator, List, Optional, Tuple
 
 import pydantic
@@ -52,12 +53,11 @@ def __init__(self, path: str, view_func: Callable) -> None:
         self.has_kwargs = False
 
         self.params = []
-        for name, arg in self.signature.parameters.items():
-            if name == "request":
-                # TODO: maybe better assert that 1st param is request or check by type?
-                # maybe even have attribute like `has_request`
-                # so that users can ignore passing request if not needed
-                continue
+        for name, arg in islice(self.signature.parameters.items(), 1, None):
+            # skip the first param since that must be the request
+            # TODO: maybe better assert that 1st param is request or check by type?
+            # maybe even have attribute like `has_request`
+            # so that users can ignore passing request if not needed
 
             if arg.kind == arg.VAR_KEYWORD:
                 # Skipping **kwargs

tests/test_signature_details.py

@@ -2,8 +2,9 @@
 from sys import version_info
 
 import pytest
+from django.http import HttpRequest
 
-from ninja.signature.details import is_collection_type
+from ninja.signature.details import ViewSignature, is_collection_type
 
 
 @pytest.mark.parametrize(
@@ -49,3 +50,19 @@
 )
 def test_is_collection_type_returns(annotation: typing.Any, expected: bool):
     assert is_collection_type(annotation) is expected
+
+
+def test_finds_request_param_by_name():
+    def view(request):
+        pass
+
+    signature = ViewSignature("", view)
+    assert signature.models == []
+
+
+def test_finds_request_param_by_type():
+    def view(_: HttpRequest):
+        pass
+
+    signature = ViewSignature("", view)
+    assert signature.models == []