From 8ecd30db0e84959b73aa9643a161274dac4af801 Mon Sep 17 00:00:00 2001
From: Esa Kataja <ekataja@gmail.com>
Date: Mon, 25 Nov 2024 19:21:03 +0200
Subject: [PATCH] Refactor Add Ruff linting options. Fix suggestions

---
 pyproject.toml               | 5 +++++
 src/lib/database/database.py | 1 -
 src/models/__init__.py       | 6 ------
 src/routes/card.py           | 2 +-
 src/routes/user.py           | 3 +--
 5 files changed, 7 insertions(+), 10 deletions(-)

diff --git a/pyproject.toml b/pyproject.toml
index f27e333..389ec9d 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -19,4 +19,9 @@ dependencies = [
 [dependency-groups]
 dev = [
     "ruff>=0.8.0",
+    
 ]
+
+[tool.ruff]
+extend-select = ["C4", "SIM", "TCH", "ERA", "FAST", "DOC"]
+target-version = "py312"
\ No newline at end of file
diff --git a/src/lib/database/database.py b/src/lib/database/database.py
index f3cbf5e..beb8e80 100644
--- a/src/lib/database/database.py
+++ b/src/lib/database/database.py
@@ -26,7 +26,6 @@ def init_db():
         from os import unlink
 
         unlink(settings.db_url)
-        # return
     sql = ""
     with open("lib/database/database.sql") as f:
         sql = f.read()
diff --git a/src/models/__init__.py b/src/models/__init__.py
index 30d1986..3b23e10 100644
--- a/src/models/__init__.py
+++ b/src/models/__init__.py
@@ -1,11 +1,5 @@
-# from .movie import Movie
-# from .score import Score
-# from .card import Card
 from .general import Message
 
 __all__ = [
-    # "Movie",
-    # "Score",
-    # "Card",
     "Message",
 ]
diff --git a/src/routes/card.py b/src/routes/card.py
index a85f376..4cad2d8 100644
--- a/src/routes/card.py
+++ b/src/routes/card.py
@@ -10,7 +10,7 @@ router = APIRouter(
 )
 
 
-@router.get("/", response_model=list[Card])
+@router.get("/", responses={200: {"model": list[Card]}})
 async def get_cards() -> list[Card]:
     """Get all cards"""
     return get_all_cards()
diff --git a/src/routes/user.py b/src/routes/user.py
index 4541cd7..21b456f 100644
--- a/src/routes/user.py
+++ b/src/routes/user.py
@@ -14,8 +14,7 @@ router = APIRouter(
 
 @router.get(
     "/",
-    response_model=list[UserOut],
-    responses={200: {"model": UserOut}, 404: {"model": Message}},
+    responses={200: {"model": list[UserOut]}, 404: {"model": Message}},
 )
 async def get_users() -> list[UserOut]:
     """Get all users"""