Fix: version variable was left undeclared in some cases in migrate()

Author: maximebf

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "sqlorm-py"
-version = "0.2.0"
+version = "0.2.1"
 description = "A new kind or ORM that do not abstract away your database or SQL queries."
 authors = ["Maxime Bouroumeau-Fuseau <[email protected]>"]
 readme = "README.md"
@@ -33,4 +33,4 @@ exclude = ["sqlorm/__init__.py"]
 
 [build-system]
 requires = ["poetry-core"]
-build-backend = "poetry.core.masonry.api"
+build-backend = "poetry.core.masonry.api"

sqlorm/schema.py

@@ -82,6 +82,7 @@ def migrate(
         if from_version is not None and logger:
             logger.info(f"Resume migrations from {from_version}")
         migrations = create_migrations_from_dir(path, from_version, to_version)
+        version = None
         for version, name, filename in migrations:
             if logger:
                 logger.info(f"Executing migration {version}: {name}")
@@ -95,7 +96,7 @@ def migrate(
                 if logger:
                     logger.error("Last migration failed, ending")
                 raise
-        if use_schema_version and not save_version_after_step:
+        if not dryrun and version is not None and use_schema_version and not save_version_after_step:
             set_schema_version(version, engine)
         return version