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 extracting, copying, and squashing commits to the rebase menu #1880

Merged
merged 8 commits into from
Apr 5, 2024

Conversation

kaste
Copy link
Collaborator

@kaste kaste commented Apr 5, 2024

👀

This probably makes the Rebase menu within the graph view feature complete to deprecate the rebase dashboard.

kaste added 8 commits April 6, 2024 00:42
Pre-fill `initial_text` if the user's selections *starts on* a (local)
branch.

This seems useful as we can put labels (branches) on commits in a
branchless or stacked workflow to keep some overview about the distinct
features, and then move them out "onto origin/main" quickly.
We have two case here.

For the copy action, the rebase would get a no-op, as we copy commits
to `ref` but then keep `ref` at the same position as before, basically
overwriting what we did.  Here name the olf ref `<ref>--old` for now.

Of course a user has then to take action and probably have to delete
that "--old" ref.  But it helps understanding the process.  Let's see
how that works out.

For the extract action, the rebase would be desastrous as we move the
commits to the ref, *drop* the original commits, and then place the old
`ref` at its old position.  So we basically just loose the commits we
wanted to extract.  Here placing a `<ref>--old` label at the old
position doesn't make sense as the old place is dropped and gone.  So
we just drop the old ref too.

🤞
@kaste kaste enabled auto-merge April 5, 2024 22:47
@kaste kaste merged commit e4662aa into master Apr 5, 2024
8 checks passed
@kaste kaste deleted the extract-and-copy-commits branch April 5, 2024 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant