fix(linter): update to make the linter happy

aerialls · Dec 19, 2020 · 726def5 · 726def5
1 parent 061ed3a
commit 726def5
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/cmd/scaleway-ddns/main.go b/cmd/scaleway-ddns/main.go
@@ -68,7 +68,10 @@ func init() {
 	rootCmd.Flags().BoolVarP(&verbose, "verbose", "v", false, "enable verbose logging")
 	rootCmd.Flags().BoolVarP(&dryRun, "dry-run", "d", false, "don't update DNS records")
 
-	rootCmd.MarkFlagRequired("config")
+	err := rootCmd.MarkFlagRequired("config")
+	if err != nil {
+		logger.Fatal(err)
+	}
 }
 
 func initConfig() {

diff --git a/cmd/scaleway-ddns/version.go b/cmd/scaleway-ddns/version.go
@@ -16,7 +16,7 @@ var versionCmd = &cobra.Command{
 	Use:   "version",
 	Short: "Prints the version",
 	Run: func(cmd *cobra.Command, args []string) {
-		fmt.Println(fmt.Sprintf("scaleway-ddns %s (commit %s, built at %s)", version, commit, date))
+		fmt.Printf("scaleway-ddns %s (commit %s, built at %s)\n", version, commit, date)
 	},
 }
 

diff --git a/ddns/ddns.go b/ddns/ddns.go
@@ -32,11 +32,8 @@ func (d *DynamicDNSUpdater) Start() {
 	d.doStart()
 
 	ticker := time.NewTicker(time.Duration(cfg.Interval) * time.Second)
-	for {
-		select {
-		case <-ticker.C:
-			d.doStart()
-		}
+	for range ticker.C {
+		d.doStart()
 	}
 }