Fixed mypy errors

Author: vodkar

.copier/.copier-answers.yml.jinja

@@ -1,4 +1,3 @@
-import os
 from logging.config import fileConfig
 
 from alembic import context
@@ -10,30 +9,21 @@
 
 # Interpret the config file for Python logging.
 # This line sets up loggers basically.
-fileConfig(config.config_file_name)
+if config.config_file_name:
+    fileConfig(config.config_file_name)
 
-# add your model's MetaData object here
-# for 'autogenerate' support
-# from myapp import mymodel
-# target_metadata = mymodel.Base.metadata
-# target_metadata = None
 
-from app.models import SQLModel  # noqa
-from app.core.config import settings # noqa
+from app.core.config import settings  # noqa
+from sqlmodel import SQLModel
 
 target_metadata = SQLModel.metadata
 
-# other values from the config, defined by the needs of env.py,
-# can be acquired:
-# my_important_option = config.get_main_option("my_important_option")
-# ... etc.
 
-
-def get_url():
+def get_url() -> str:
     return str(settings.SQLALCHEMY_DATABASE_URI)
 
 
-def run_migrations_offline():
+def run_migrations_offline() -> None:
     """Run migrations in 'offline' mode.
 
     This configures the context with just a URL
@@ -47,21 +37,24 @@ def run_migrations_offline():
     """
     url = get_url()
     context.configure(
-        url=url, target_metadata=target_metadata, literal_binds=True, compare_type=True
+        url=url,
+        target_metadata=target_metadata,
+        literal_binds=True,
+        compare_type=True,
     )
 
     with context.begin_transaction():
         context.run_migrations()
 
 
-def run_migrations_online():
+def run_migrations_online() -> None:
     """Run migrations in 'online' mode.
 
     In this scenario we need to create an Engine
     and associate a connection with the context.
 
     """
-    configuration = config.get_section(config.config_ini_section)
+    configuration = config.get_section(config.config_ini_section) or {}
     configuration["sqlalchemy.url"] = get_url()
     connectable = engine_from_config(
         configuration,
@@ -71,7 +64,9 @@ def run_migrations_online():
 
     with connectable.connect() as connection:
         context.configure(
-            connection=connection, target_metadata=target_metadata, compare_type=True
+            connection=connection,
+            target_metadata=target_metadata,
+            compare_type=True,
         )
 
         with context.begin_transaction():

.copier/update_dotenv.py

@@ -5,33 +5,30 @@
 Create Date: 2024-07-31 22:24:34.447891
 
 """
-from alembic import op
-import sqlalchemy as sa
-import sqlmodel.sql.sqltypes
 
+import sqlalchemy as sa
+from alembic import op
 
 # revision identifiers, used by Alembic.
-revision = '1a31ce608336'
-down_revision = 'd98dd8ec85a3'
+revision = "1a31ce608336"
+down_revision = "d98dd8ec85a3"
 branch_labels: str | None = None
 depends_on: str | None = None
 
 
 def upgrade() -> None:
     # ### commands auto generated by Alembic - please adjust! ###
-    op.alter_column('item', 'owner_id',
-               existing_type=sa.UUID(),
-               nullable=False)
-    op.drop_constraint('item_owner_id_fkey', 'item', type_='foreignkey')
-    op.create_foreign_key(None, 'item', 'user', ['owner_id'], ['id'], ondelete='CASCADE')
+    op.alter_column("item", "owner_id", existing_type=sa.UUID(), nullable=False)
+    op.drop_constraint("item_owner_id_fkey", "item", type_="foreignkey")
+    op.create_foreign_key(
+        None, "item", "user", ["owner_id"], ["id"], ondelete="CASCADE",
+    )
     # ### end Alembic commands ###
 
 
 def downgrade() -> None:
     # ### commands auto generated by Alembic - please adjust! ###
-    op.drop_constraint('item_owner_id_fkey', 'item', type_='foreignkey')
-    op.create_foreign_key('item_owner_id_fkey', 'item', 'user', ['owner_id'], ['id'])
-    op.alter_column('item', 'owner_id',
-               existing_type=sa.UUID(),
-               nullable=True)
+    op.drop_constraint("item_owner_id_fkey", "item", type_="foreignkey")
+    op.create_foreign_key("item_owner_id_fkey", "item", "user", ["owner_id"], ["id"])
+    op.alter_column("item", "owner_id", existing_type=sa.UUID(), nullable=True)
     # ### end Alembic commands ###

backend/app/alembic/env.py

@@ -5,65 +5,88 @@
 Create Date: 2024-06-17 14:42:44.639457
 
 """
-from alembic import op
-import sqlalchemy as sa
-import sqlmodel.sql.sqltypes
 
+import sqlalchemy as sa
+from alembic import op
 
 # revision identifiers, used by Alembic.
-revision = '9c0a54914c78'
-down_revision = 'e2412789c190'
+revision = "9c0a54914c78"
+down_revision = "e2412789c190"
 branch_labels: str | None = None
 depends_on: str | None = None
 
 
 def upgrade() -> None:
     # Adjust the length of the email field in the User table
-    op.alter_column('user', 'email',
-               existing_type=sa.String(),
-               type_=sa.String(length=255),
-               existing_nullable=False)
+    op.alter_column(
+        "user",
+        "email",
+        existing_type=sa.String(),
+        type_=sa.String(length=255),
+        existing_nullable=False,
+    )
 
     # Adjust the length of the full_name field in the User table
-    op.alter_column('user', 'full_name',
-               existing_type=sa.String(),
-               type_=sa.String(length=255),
-               existing_nullable=True)
+    op.alter_column(
+        "user",
+        "full_name",
+        existing_type=sa.String(),
+        type_=sa.String(length=255),
+        existing_nullable=True,
+    )
 
     # Adjust the length of the title field in the Item table
-    op.alter_column('item', 'title',
-               existing_type=sa.String(),
-               type_=sa.String(length=255),
-               existing_nullable=False)
+    op.alter_column(
+        "item",
+        "title",
+        existing_type=sa.String(),
+        type_=sa.String(length=255),
+        existing_nullable=False,
+    )
 
     # Adjust the length of the description field in the Item table
-    op.alter_column('item', 'description',
-               existing_type=sa.String(),
-               type_=sa.String(length=255),
-               existing_nullable=True)
+    op.alter_column(
+        "item",
+        "description",
+        existing_type=sa.String(),
+        type_=sa.String(length=255),
+        existing_nullable=True,
+    )
 
 
 def downgrade() -> None:
     # Revert the length of the email field in the User table
-    op.alter_column('user', 'email',
-               existing_type=sa.String(length=255),
-               type_=sa.String(),
-               existing_nullable=False)
+    op.alter_column(
+        "user",
+        "email",
+        existing_type=sa.String(length=255),
+        type_=sa.String(),
+        existing_nullable=False,
+    )
 
     # Revert the length of the full_name field in the User table
-    op.alter_column('user', 'full_name',
-               existing_type=sa.String(length=255),
-               type_=sa.String(),
-               existing_nullable=True)
+    op.alter_column(
+        "user",
+        "full_name",
+        existing_type=sa.String(length=255),
+        type_=sa.String(),
+        existing_nullable=True,
+    )
 
     # Revert the length of the title field in the Item table
-    op.alter_column('item', 'title',
-               existing_type=sa.String(length=255),
-               type_=sa.String(),
-               existing_nullable=False)
+    op.alter_column(
+        "item",
+        "title",
+        existing_type=sa.String(length=255),
+        type_=sa.String(),
+        existing_nullable=False,
+    )
 
     # Revert the length of the description field in the Item table
-    op.alter_column('item', 'description',
-               existing_type=sa.String(length=255),
-               type_=sa.String(),
-               existing_nullable=True)
+    op.alter_column(
+        "item",
+        "description",
+        existing_type=sa.String(length=255),
+        type_=sa.String(),
+        existing_nullable=True,
+    )