Swallow errors entirely, instead of assigning and ignoring them

This commit is contained in:
Christian Muehlhaeuser 2019-07-19 03:55:33 +02:00 committed by Jesse Duffield
parent 975a5315b0
commit 69ac0036e6
2 changed files with 3 additions and 5 deletions

View file

@ -261,10 +261,8 @@ func (c *CommitListBuilder) getMergeBase() (string, error) {
baseBranch = "develop" 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 return output, nil
} }

View file

@ -29,7 +29,7 @@ func main() {
splitVersion := strings.Split(stringVersion, ".") splitVersion := strings.Split(stringVersion, ".")
patch := splitVersion[len(splitVersion)-1] patch := splitVersion[len(splitVersion)-1]
newPatch, err := strconv.Atoi(patch) newPatch, _ := strconv.Atoi(patch)
splitVersion[len(splitVersion)-1] = strconv.FormatInt(int64(newPatch)+1, 10) splitVersion[len(splitVersion)-1] = strconv.FormatInt(int64(newPatch)+1, 10)
newVersion := strings.Join(splitVersion, ".") newVersion := strings.Join(splitVersion, ".")