search

wfxr / forgit

:zzz: A utility tool powered by fzf for using git interactively.
MIT License
4.4k stars 137 forks source link

Checkout file from branch & stash push #157

Closed D4KU closed 2 years ago

D4KU commented 3 years ago

Check list

Description

Added two new bash/zsh commands to:

I didn't add them to the readme. My thinking was that it's probably better to document them only when implemented in fish.

Type of change

Test environment

cjappl commented 3 years ago

I would recommend documenting them as a part of this PR. It helps me test when I’m implementing them in fish!

On Wed, Jul 7, 2021 at 12:23 PM, David Kutschke @.***> wrote:

Check list

  • I have performed a self-review of my code
  • I have commented my code in hard-to-understand areas
  • I have made corresponding changes to the documentation

Description

Added two new bash/zsh commands to:

  • checkout selected files from given branch
  • stash selected files

I didn't add them to the readme. My thinking was that it's probably better to document them only when implemented in fish.

Type of change

  • Bug fix
  • New feature
  • Refactor
  • Breaking change
  • Documentation change

Test environment

  • Shell

  • bash

  • zsh

  • fish

  • OS

  • Linux

  • Mac OS X

  • Windows

  • Others:

You can view, comment on, or merge this pull request online at:

https://github.com/wfxr/forgit/pull/157

Commit Summary

  • Added command to checkout file from branch
  • Added command to push files on stash
  • Simplified checkout file from branch

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

D4KU commented 3 years ago

That makes sense. I added them to the readme.

cjappl commented 3 years ago

@wfxr pinging for a review when you get a chance

cjappl commented 3 years ago

@wfxr I would say I support this coming in. Especially if @D4KU had time to add a git stash apply or git stash show for feature symmetry.

Hold off until @wfxr approves, just to be sure we want to take this. I give this change a big thumbs up, though :)

D4KU commented 3 years ago

Especially if @D4KU had time to add a git stash apply or git stash show for feature symmetry.

No problem at all. :)

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.