drewdeponte / git-ps-rs

Official git-ps Rust implementation - the future of git-ps
https://git-ps.sh
MIT License
78 stars 8 forks source link

List importable branches/patches #48

Open drewdeponte opened 2 years ago

drewdeponte commented 2 years ago

One thought I had as I was thinking about the idea of importing patches from remote branches into the patch stack was seeing if there is a way to hide the concept of the branches.

Maybe there is a command that gps import ls or something that lists all the available patches that can be imported.

If we are talking patches it would list out each of the commits from the remote branches that don't already exist in your patch stack.

Then there could be another command that would gps import <patch-index> or something that would do the actual importing of that patch into your current patch stack by cherry-picking that patch onto the top of your patch stack.

ctsstc commented 2 years ago

I like this idea of displaying "remote patches".

I think you would potentially want to still keep them separated in the patch stack as something like "remote integrated patches" or "checked out patches" once pulling them down and in to your patch stack.

I like the idea and verbiage of "checking out a patch" and potentially removing the checked out patch after you're done looking at it. Or would you expect folks to keep the remote patch integrated until it eventually hits the main branch and clears out?

I think that you would also need to have a status on the patch list that shows if the remote patch has changed thus making the local patch "dirty"/out of sync.