Add --splice support to merge command#1757
Conversation
sqla-tester
left a comment
There was a problem hiding this comment.
OK, this is sqla-tester setting up my work on behalf of CaselIT to try to get revision 52bf8a6 of this pull request into gerrit so we can run tests and reviews and stuff
|
New Gerrit review created for change 52bf8a6: https://gerrit.sqlalchemy.org/c/sqlalchemy/alembic/+/6496 |
|
Michael Bayer (zzzeek) wrote: hi - sorry, another PR for the same feature came along and I forgot that this one was already in the queue from a few months back, this one wasn't passing CI for wahtever reason. thanks for the submission though View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/alembic/+/6496 |
|
Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/alembic/+/6496 has been abandoned. That means that at least for the moment I need to close this pull request. Sorry it didn't work out :( |
This fixes: #1712
Description
I noticed that no one was actively working on this issue, so I went ahead and implemented support for the --splice option in the merge command.
This change allows merge to operate with non-head revisions by disabling the head-only validation, consistent with the behavior of --splice in revision.
Key points included in this PR:
Let me know if further adjustments or additional tests would be helpful.
Checklist
This pull request is:
must include a complete example of the issue. one line code fixes without an
issue and demonstration will not be accepted.
Fixes: #<issue number>in the commit messageinclude a complete example of how the feature would look.
Fixes: #<issue number>in the commit messageHave a nice day!