Skip to content

Ranged copy hash #1244

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

Closed
intrnl opened this issue Jun 24, 2022 · 6 comments · Fixed by #1288
Closed

Ranged copy hash #1244

intrnl opened this issue Jun 24, 2022 · 6 comments · Fixed by #1288
Labels
Milestone

Comments

@intrnl
Copy link

intrnl commented Jun 24, 2022

Is your feature request related to a problem? Please describe.
Cherry picking is a daily occurence for my work, and it would usually involve a lot of commits needing to be picked, often 1, but often 10-20 commits as well.

Describe the solution you'd like
It would be nice if we can do a commit selection and have the copy hash shortcut copy the range.

Describe alternatives you've considered
Yes, I could just do a copy and paste of every single commits I want picked, however that's not fast and convenient.

Additional context
None.

@extrawurst extrawurst added the good first issue Good for newcomers label Aug 17, 2022
@extrawurst
Copy link
Collaborator

do you want all the commits listed in a comma-separated way or some <commithash1>..<commithash2> notation?

@intrnl
Copy link
Author

intrnl commented Aug 17, 2022

The former is okay, the latter is a nice topping on top.

@extrawurst
Copy link
Collaborator

well the one is fundamentally different to the other :)
one can have gaps, the other not. so I lean towards the comma-separated list

@remique
Copy link
Contributor

remique commented Aug 18, 2022

IMO reasonable approach would be space-separated commits unless they are selected in turn. This would allow both git cherry-pick c1 c3 c5 and git cherry-pick c1^..c5.

@extrawurst
Copy link
Collaborator

@remique fine by me. are you up for making it a PR? ;)

@remique
Copy link
Contributor

remique commented Aug 18, 2022

Sure, working on it 👍

@extrawurst extrawurst added this to the v0.22 milestone Aug 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants