Skip to content

Commit 2a4b850

Browse files
authored
Merge pull request #4657 from kersten/fix/version-parsing-avoid-shadowing-err
🌱 (chore): avoid shadowing of 'err' in version parsing for plugin and config packages
2 parents 1ec455e + 85cd43e commit 2a4b850

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

pkg/config/version.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ func (v *Version) Parse(version string) error {
5151
var err error
5252
if v.Number, err = strconv.Atoi(substrings[0]); err != nil {
5353
// Let's check if the `-` belonged to a negative number
54-
if n, err := strconv.Atoi(version); err == nil && n < 0 {
54+
if n, errParse := strconv.Atoi(version); errParse == nil && n < 0 {
5555
return errNonPositive
5656
}
5757
return err

pkg/plugin/version.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ func (v *Version) Parse(version string) error {
5151
var err error
5252
if v.Number, err = strconv.Atoi(substrings[0]); err != nil {
5353
// Let's check if the `-` belonged to a negative number
54-
if n, err := strconv.Atoi(version); err == nil && n < 0 {
54+
if n, errParse := strconv.Atoi(version); errParse == nil && n < 0 {
5555
return errNegative
5656
}
5757
return err

0 commit comments

Comments
 (0)