1
0
mirror of https://github.com/jesseduffield/lazygit.git synced 2025-06-08 23:56:15 +02:00

Improve prompts when amending commits (#2980)

This commit is contained in:
Stefan Haller 2023-09-04 09:45:34 +02:00 committed by GitHub
commit e2c447b8d0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
5 changed files with 29 additions and 27 deletions

View File

@ -615,6 +615,10 @@ func (self *FilesController) refresh() error {
} }
func (self *FilesController) handleAmendCommitPress() error { func (self *FilesController) handleAmendCommitPress() error {
return self.c.Confirm(types.ConfirmOpts{
Title: self.c.Tr.AmendLastCommitTitle,
Prompt: self.c.Tr.SureToAmend,
HandleConfirm: func() error {
return self.c.Helpers().WorkingTree.WithEnsureCommitableFiles(func() error { return self.c.Helpers().WorkingTree.WithEnsureCommitableFiles(func() error {
if len(self.c.Model().Commits) == 0 { if len(self.c.Model().Commits) == 0 {
return self.c.ErrorMsg(self.c.Tr.NoCommitToAmend) return self.c.ErrorMsg(self.c.Tr.NoCommitToAmend)
@ -622,6 +626,8 @@ func (self *FilesController) handleAmendCommitPress() error {
return self.c.Helpers().AmendHelper.AmendHead() return self.c.Helpers().AmendHelper.AmendHead()
}) })
},
})
} }
func (self *FilesController) handleStatusFilterPressed() error { func (self *FilesController) handleStatusFilterPressed() error {

View File

@ -1,9 +1,5 @@
package helpers package helpers
import (
"github.com/jesseduffield/lazygit/pkg/gui/types"
)
type AmendHelper struct { type AmendHelper struct {
c *HelperCommon c *HelperCommon
gpg *GpgHelper gpg *GpgHelper
@ -20,13 +16,7 @@ func NewAmendHelper(
} }
func (self *AmendHelper) AmendHead() error { func (self *AmendHelper) AmendHead() error {
return self.c.Confirm(types.ConfirmOpts{
Title: self.c.Tr.AmendLastCommitTitle,
Prompt: self.c.Tr.SureToAmend,
HandleConfirm: func() error {
cmdObj := self.c.Git().Commit.AmendHeadCmdObj() cmdObj := self.c.Git().Commit.AmendHeadCmdObj()
self.c.LogAction(self.c.Tr.Actions.AmendCommit) self.c.LogAction(self.c.Tr.Actions.AmendCommit)
return self.gpg.WithGpgHandling(cmdObj, self.c.Tr.AmendingStatus, nil) return self.gpg.WithGpgHandling(cmdObj, self.c.Tr.AmendingStatus, nil)
},
})
} }

View File

@ -568,12 +568,18 @@ func (self *LocalCommitsController) moveUp(commit *models.Commit) error {
func (self *LocalCommitsController) amendTo(commit *models.Commit) error { func (self *LocalCommitsController) amendTo(commit *models.Commit) error {
if self.isHeadCommit() { if self.isHeadCommit() {
return self.c.Confirm(types.ConfirmOpts{
Title: self.c.Tr.AmendCommitTitle,
Prompt: self.c.Tr.AmendCommitPrompt,
HandleConfirm: func() error {
return self.c.Helpers().WorkingTree.WithEnsureCommitableFiles(func() error { return self.c.Helpers().WorkingTree.WithEnsureCommitableFiles(func() error {
if err := self.c.Helpers().AmendHelper.AmendHead(); err != nil { if err := self.c.Helpers().AmendHelper.AmendHead(); err != nil {
return err return err
} }
return self.c.Refresh(types.RefreshOptions{Mode: types.ASYNC}) return self.c.Refresh(types.RefreshOptions{Mode: types.ASYNC})
}) })
},
})
} }
if self.c.Git().Status.WorkingTreeState() != enums.REBASE_MODE_NONE { if self.c.Git().Status.WorkingTreeState() != enums.REBASE_MODE_NONE {

View File

@ -43,8 +43,8 @@ var AmendHeadCommitDuringRebase = NewIntegrationTest(NewIntegrationTestArgs{
Press(keys.Commits.AmendToCommit). Press(keys.Commits.AmendToCommit).
Tap(func() { Tap(func() {
t.ExpectPopup().Confirmation(). t.ExpectPopup().Confirmation().
Title(Equals("Amend last commit")). Title(Equals("Amend commit")).
Content(Contains("Are you sure you want to amend last commit?")). Content(Contains("Are you sure you want to amend this commit with your staged files?")).
Confirm() Confirm()
}). }).
Lines( Lines(

View File

@ -42,8 +42,8 @@ var AmendMerge = NewIntegrationTest(NewIntegrationTestArgs{
Press(keys.Commits.AmendToCommit) Press(keys.Commits.AmendToCommit)
t.ExpectPopup().Confirmation(). t.ExpectPopup().Confirmation().
Title(Equals("Amend last commit")). Title(Equals("Amend commit")).
Content(Contains("Are you sure you want to amend last commit?")). Content(Contains("Are you sure you want to amend this commit with your staged files?")).
Confirm() Confirm()
// assuring we haven't added a brand new commit // assuring we haven't added a brand new commit