-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
3533 Change reset
terminal-run commands into normal commands w/ proper error handling
#3676
base: main
Are you sure you want to change the base?
3533 Change reset
terminal-run commands into normal commands w/ proper error handling
#3676
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
e6953f6
to
94bda2c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested with "reset current branch to tip" graph context menu command, and it LGTM. Let me know if I should test other cases.
3adc472
to
1a652e4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sergiolms I went ahead and fixed the ☝🏼 error |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I couldn't reproduce any of the error types here, but I don't have further time to test. @eamodio feel free to give this a test run, otherwise LGTM
1187eac
to
cbef015
Compare
cbef015
to
e6634de
Compare
e6634de
to
7e6d1cf
Compare
Description
This task creates the
git reset
commands to move away fromrunTerminalCommand
. Solves #3533To be defined:
Checklist
Fixes $XXX -
orCloses #XXX -
prefix to auto-close the issue that your PR addresses