Skip to content

Commit

Permalink
Merge pull request #9852 from prezha/fix-update-scripts
Browse files Browse the repository at this point in the history
fix GHReleases() to use rls tag name instead of rls name for rls version
  • Loading branch information
medyagh authored Dec 4, 2020
2 parents 9c3c127 + bbd0529 commit a6d56c6
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion hack/update/github.go
Original file line number Diff line number Diff line change
Expand Up @@ -221,7 +221,7 @@ func GHReleases(ctx context.Context, owner, repo string) (stable, latest string,
return "", "", err
}
for _, rl := range rls {
ver := rl.GetName()
ver := rl.GetTagName()
if !semver.IsValid(ver) {
continue
}
Expand Down
3 changes: 1 addition & 2 deletions hack/update/update.go
Original file line number Diff line number Diff line change
Expand Up @@ -88,8 +88,7 @@ func (i *Item) apply(data interface{}) error {
if i.Content == nil {
return fmt.Errorf("unable to update content: nothing to update")
}
org := string(i.Content)
str := org
str := string(i.Content)
for src, dst := range i.Replace {
out, err := ParseTmpl(dst, data, "")
if err != nil {
Expand Down

0 comments on commit a6d56c6

Please sign in to comment.