Throw an exception if create_database_migration_numbered_style fails

martinburchell · martinburchell · commit 62ae8611d4c9 · 2025-03-11T11:45:18.000Z
diff --git a/cardinal_pythonlib/sqlalchemy/alembic_func.py b/cardinal_pythonlib/sqlalchemy/alembic_func.py
@@ -400,7 +400,7 @@ def create_database_migration_numbered_style(
         new_seq_str,
     ]
     log.info("From directory {!r}, calling: {!r}", alembic_ini_dir, cmdargs)
-    subprocess.call(cmdargs)
+    subprocess.run(cmdargs, check=True)
 
 
 def stamp_allowing_unusual_version_table(

Original file line number	Diff line number	Diff line change
`@@ -400,7 +400,7 @@ def create_database_migration_numbered_style(`
`400`	`400`	`new_seq_str,`
`401`	`401`	`]`
`402`	`402`	`log.info("From directory {!r}, calling: {!r}", alembic_ini_dir, cmdargs)`
`403`		`- subprocess.call(cmdargs)`
	`403`	`+ subprocess.run(cmdargs, check=True)`
`404`	`404`
`405`	`405`
`406`	`406`	`def stamp_allowing_unusual_version_table(`