-
Notifications
You must be signed in to change notification settings - Fork 318
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
rebase: allow -r
to accept multiple revisions
#3564
Conversation
How much will this conflict with #3396 ( EDIT: "because it seems important to keep it in mind when designing" can of course apply in the other direction too, but I think it seems like a bigger risk that |
Agreed, I would rather this PR get merged first. It’s functionally ready, but I probably need to do some minor code cleanup as well. |
030e317
to
e557a66
Compare
940ac95
to
a34b7e0
Compare
7119397
to
679ce3f
Compare
679ce3f
to
4968bab
Compare
837a5b9
to
5389acf
Compare
5389acf
to
6814dea
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
Closes #1158.
Checklist
If applicable:
CHANGELOG.md