diff options
author | Christian Muehlhaeuser <muesli@gmail.com> | 2019-07-19 03:55:33 +0200 |
---|---|---|
committer | Jesse Duffield <jessedduffield@gmail.com> | 2019-07-27 10:53:19 +1000 |
commit | 69ac0036e67ac22fdf3baaa083a9020d001180d6 (patch) | |
tree | e36d5f07a8d82a40693772fcbb6717700361e52e /pkg/git/commit_list_builder.go | |
parent | 975a5315b036b84e0d8befd7ca5e7dbc062caf84 (diff) |
Swallow errors entirely, instead of assigning and ignoring them
Diffstat (limited to 'pkg/git/commit_list_builder.go')
-rw-r--r-- | pkg/git/commit_list_builder.go | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/pkg/git/commit_list_builder.go b/pkg/git/commit_list_builder.go index 60ac22f4a..a47fcbfe2 100644 --- a/pkg/git/commit_list_builder.go +++ b/pkg/git/commit_list_builder.go @@ -261,10 +261,8 @@ func (c *CommitListBuilder) getMergeBase() (string, error) { baseBranch = "develop" } - output, err := c.OSCommand.RunCommandWithOutput(fmt.Sprintf("git merge-base HEAD %s", baseBranch)) - if err != nil { - // swallowing error because it's not a big deal; probably because there are no commits yet - } + // swallowing error because it's not a big deal; probably because there are no commits yet + output, _ := c.OSCommand.RunCommandWithOutput(fmt.Sprintf("git merge-base HEAD %s", baseBranch)) return output, nil } |