go/lazygit
1
0
Fork 0
mirror of https://github.com/jesseduffield/lazygit.git synced 2024-11-26 09:00:57 +02:00
Code Issues Releases Activity

Remove unnecessary nil error in NewPullRequest.

Browse Source
This commit is contained in:
Kristijan Husak 2018-10-15 11:00:19 +02:00
parent df0e3e52fe
commit c69fce2e9d
3 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
pkg/commands/pull_request.go
View File

@ -44,11 +44,11 @@ func getServices() []*Service {
}
// NewPullRequest creates new instance of PullRequest
func NewPullRequest(gitCommand *GitCommand) (*PullRequest, error) {
func NewPullRequest(gitCommand *GitCommand) *PullRequest {
return &PullRequest{
GitServices: getServices(),
GitCommand: gitCommand,
}, nil
}
}
// Create opens link to new pull request in browser

5
pkg/commands/pull_request_test.go
View File

@ -1,10 +1,11 @@
package commands
import (
"github.com/stretchr/testify/assert"
"os/exec"
"strings"
"testing"
"github.com/stretchr/testify/assert"
)
func TestGetRepoInfoFromURL(t *testing.T) {
@ -144,7 +145,7 @@ func TestCreatePullRequest(t *testing.T) {
gitCommand := newDummyGitCommand()
gitCommand.OSCommand.command = s.command
gitCommand.OSCommand.Config.GetUserConfig().Set("os.openCommand", "open {{filename}}")
dummyPullRequest, _ := NewPullRequest(gitCommand)
dummyPullRequest := NewPullRequest(gitCommand)
s.test(dummyPullRequest.Create(s.branch))
})
}

2
pkg/gui/branches_panel.go
View File

@ -24,7 +24,7 @@ func (gui *Gui) handleBranchPress(g *gocui.Gui, v *gocui.View) error {
func (gui *Gui) handleCreatePullRequestPress(g *gocui.Gui, v *gocui.View) error {
branch := gui.getSelectedBranch(gui.getBranchesView(g))
pullRequest, _ := commands.NewPullRequest(gui.GitCommand)
pullRequest := commands.NewPullRequest(gui.GitCommand)
if err := pullRequest.Create(branch); err != nil {
return gui.createErrorPanel(g, err.Error())