Public/fastapi-cache
1
0
Fork 0
mirror of https://github.com/long2ice/fastapi-cache.git synced 2026-03-25 04:57:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

Correct type hint: namespace is not optional

Browse Source 
The namespace argument is positional and will never be `None` so should
not be marked as Optional. It is always a string, and the default is
to pass in an empty string.
This commit is contained in:
Martijn Pieters
2023-04-27 16:11:59 +01:00
parent ee58f979d4
commit 32acafa5e0
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
README.md
View File

@@ -121,7 +121,7 @@ take effect globally.
```python ```python
def my_key_builder( def my_key_builder(
func, func,
namespace: Optional[str] = "", namespace: str = "",
request: Request = None, request: Request = None,
response: Response = None, response: Response = None,
*args, *args,

2
fastapi_cache/key_builder.py
View File

@@ -7,7 +7,7 @@ from starlette.responses import Response
def default_key_builder( def default_key_builder(
func: Callable, func: Callable,
namespace: Optional[str] = "", namespace: str = "",
request: Optional[Request] = None, request: Optional[Request] = None,
response: Optional[Response] = None, response: Optional[Response] = None,
args: Optional[tuple] = None, args: Optional[tuple] = None,