Skip to content

Commit

Permalink
fix(compute/serve): don't fail the serve workflow if github errors
Browse files Browse the repository at this point in the history
  • Loading branch information
Integralist committed Jul 11, 2024
1 parent c2f28dc commit 75222aa
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions pkg/commands/compute/serve.go
Original file line number Diff line number Diff line change
Expand Up @@ -476,7 +476,9 @@ func (c *ServeCommand) InstallViceroy(
// IMPORTANT: We declare separately so to shadow `err` from parent scope.
var latestVersion string

err = spinner.Process("Checking latest Viceroy release", func(_ *text.SpinnerWrapper) error {
// NOTE: We won't stop the user because although we can't request the latest
// version of the tool, the user may have a local version already installed.
_ = spinner.Process("Checking latest Viceroy release", func(_ *text.SpinnerWrapper) error {
latestVersion, err = c.ViceroyVersioner.LatestVersion()
if err != nil {
return fsterr.RemediationError{
Expand All @@ -486,9 +488,6 @@ func (c *ServeCommand) InstallViceroy(
}
return nil
})
if err != nil {
return err
}

viceroyConfig := c.Globals.Config.Viceroy
viceroyConfig.LatestVersion = latestVersion
Expand Down

0 comments on commit 75222aa

Please sign in to comment.