Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add upgrade message to each step #115

Closed
sqlalchemy-bot opened this issue Mar 28, 2013 · 5 comments
Closed

Add upgrade message to each step #115

sqlalchemy-bot opened this issue Mar 28, 2013 · 5 comments

Comments

@sqlalchemy-bot
Copy link

@sqlalchemy-bot sqlalchemy-bot commented Mar 28, 2013

Migrated issue, originally created by Iuri de Silvio (@iurisilvio)

I have no idea what exactly it is running without history command. I expect the message in each migration step.

I have this:

INFO  [alembic.migration] Context impl SQLiteImpl.
INFO  [alembic.migration] Will assume transactional DDL.
INFO  [alembic.migration] Running upgrade None -> 2ff0d413f670
INFO  [alembic.migration] Running upgrade 2ff0d413f670 -> 34f667a958f5
INFO  [alembic.migration] Running upgrade 34f667a958f5 -> 39c922e01e0

It can be improved with a (message) at the end of each line, like in history command.

INFO  [alembic.migration] Running upgrade 34f667a958f5 -> 39c922e01e0 (Some message here)
@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented Mar 28, 2013

Michael Bayer (@zzzeek) wrote:

send me a pull request, easy enough

@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented Mar 30, 2013

Hong Minhee (@dahlia) wrote:

I will work on this.

@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented Mar 30, 2013

Hong Minhee (@dahlia) wrote:

I made a patch for this. See the pull request #30.

@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented Apr 4, 2013

Michael Bayer (@zzzeek) wrote:

thanks !

@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented Apr 4, 2013

Changes by Michael Bayer (@zzzeek):

  • changed status to closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.