Closed GeorchW closed 6 months ago
thanks, that change looks good.
I'll add a changelog in gerrit and after review it should be merged. Thanks!
New Gerrit review created for change 69959241d491d9e08c4fb0bc38328a232e89811b: https://gerrit.sqlalchemy.org/c/sqlalchemy/alembic/+/5105
Michael Bayer (zzzeek) wrote:
thank you!
View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/alembic/+/5105
Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/alembic/+/5105 has been merged. Congratulations! :)
Fixes #1391
There's already a function that normalizes expressions from generated columns and server defaults. It looks like this:
I added the characters
\r\n\t
to the ones that are removed.Description
I added the characters
\r\n\t
to the regex that normalizes the expressions, that's it. Also added a test case.Checklist
This pull request is:
Fixes: #<issue number>
in the commit messageFixes: #<issue number>
in the commit messageHave a nice day!