Allow db_url parameter to create_database_migration_numbered_style#32
Allow db_url parameter to create_database_migration_numbered_style#32RudolfCardinal merged 2 commits intomasterfrom
Conversation
|
If you're happy, I'll (a) merge this and push to PyPI, (b) complete the changes to ucam-department-of-psychiatry/camcops#388 with a proper cardinal_pythonlib version number. |
|
Unfortunately we've duplicated some effort here but it might that we can combine our efforts. master...throw-exception-if-migration-fails is perhaps a little less ugly. I infer from sqlalchemy/alembic#1089 that it is OK to call the command API directly, though specifying the autogenerate flag again in the additional How about we adopt my solution but keep your other cosmetic changes (though the release date in the changelog appears to have gone back in time). If we wanted to stick to calling alembic through |
|
Yours is much nicer. Yes, if we can use the API directly that is far preferable, and allows a much better db_url tweak. |
Part of fixing the
missing-migration-check.ymlproblem in ucam-department-of-psychiatry/camcops#388