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 "Reset branch to here" to context menu for Gerrit search results #414

Open
brownian-motion opened this issue Jun 28, 2021 · 0 comments

Comments

@brownian-motion
Copy link

Proposed enhancement;

Adding an entry to "reset current branch to here" to Gerrit entries, to provide an alternative to checking out or cherry picking a patch-set. This is a feature available on the normal Git log tab in IntelliJ, and would be similarly useful to users of Gerrit:
image
This command would fit well in this context menu, perhaps after the "Cherry-Pick (No Commit)" option:
image

Justification

Use case 1

When asynchronously collaborating with a teammate on one feature, especially in the case where one of us resumes work while the other is on days off, it's convenient for us to check out the other's work to continue. For example, they upload p1, I upload p2 the next day, they upload p3 the next day, I download p3 and upload p4 the next day. Since I already have a branch for this change, resetting would let me re-use my branch without having to fall back to the command line or messing with branches myself to continue work on the current branch.

Use case 2

Occasionally, the juniors at my company make a mistake, push something incorrect, and want to reset back to an earlier patch set. Being able to reset to an old patch set from their IDE would simplify their approach to resolve that sort of problem. One possible implementation would be to let them select the patch set using the dropdown on the right of each entry, and then select "Reset to here" from the context menu. An alternative UI to reset to a particular patch-set may be to provide context menu items for the pane listing the patch sets:
image

brownian-motion added a commit to brownian-motion/gerrit-intellij-plugin that referenced this issue Jul 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants