diff --git a/cli/common/hooks.go b/cli/common/hooks.go index 5febcfa13..952742cba 100644 --- a/cli/common/hooks.go +++ b/cli/common/hooks.go @@ -57,7 +57,7 @@ func After(_ *cli.Context) error { if waitForUpdateCheck != nil { select { case <-waitForUpdateCheck.Done(): - // When the actual command already finished, we still wait 250ms for the update check to finish + // When the actual command already finished, we still wait 500ms for the update check to finish case <-time.After(time.Millisecond * 500): log.Debug().Msg("Update check stopped due to timeout") cancelWaitForUpdate(errors.New("update check timeout")) diff --git a/cli/internal/config/config.go b/cli/internal/config/config.go index 77405dc66..50d32bec0 100644 --- a/cli/internal/config/config.go +++ b/cli/internal/config/config.go @@ -30,9 +30,12 @@ func Load(c *cli.Context) error { return err } - if config == nil && !c.IsSet("server-url") && !c.IsSet("token") { - log.Info().Msg("The woodpecker-cli is not yet set up. Please run `woodpecker-cli setup`") - return errors.New("woodpecker-cli is not setup") + if config == nil { + config = &Config{ + LogLevel: "info", + ServerURL: c.String("server-url"), + Token: c.String("token"), + } } if !c.IsSet("server") { @@ -56,6 +59,11 @@ func Load(c *cli.Context) error { } } + if config.ServerURL == "" || config.Token == "" { + log.Info().Msg("The woodpecker-cli is not yet set up. Please run `woodpecker-cli setup` or provide the required environment variables / flags.") + return errors.New("woodpecker-cli is not configured") + } + return nil }