style: style & refactor

Author: wandapeter

cmd/disable.go

@@ -5,7 +5,6 @@ import (
 	"github.com/spf13/cobra"
 )
 
-// enableCmd represents the enable command
 var disableCmd = &cobra.Command{
 	Use:   "disable",
 	Short: "Disable the copr repository",

cmd/enable.go

@@ -5,7 +5,6 @@ import (
 	"github.com/spf13/cobra"
 )
 
-// enableCmd represents the enable command
 var enableCmd = &cobra.Command{
 	Use:   "enable",
 	Short: "Enable the copr repository",

cmd/root.go

@@ -17,8 +17,6 @@ var rootCmd = &cobra.Command{
 	Long:    `rpm-copr is a Command Line Interface that ports the COPR dnf plugin to immutable (OSTree) images.`,
 }
 
-// Execute adds all child commands to the root command and sets flags appropriately.
-// This is called by main.main(). It only needs to happen once to the rootCmd.
 func Execute() {
 	err := rootCmd.Execute()
 	if err != nil {

go.mod

@@ -5,5 +5,5 @@ go 1.21.4
 require (
 	github.com/inconshreveable/mousetrap v1.1.0 // indirect
 	github.com/spf13/cobra v1.8.0
-	github.com/spf13/pflag v1.0.5 // indirect
+	github.com/spf13/pflag v1.0.5
 )

main.go

@@ -1,7 +1,3 @@
-/*
-Copyright © 2023 NAME HERE <EMAIL ADDRESS>
-
-*/
 package main
 
 import "github.com/34N0/rpm-copr/cmd"

pkg/repos/copr.go

@@ -77,6 +77,7 @@ func (c Copr) getRepoConfig() string {
 	return string(body)
 }
 
+// red the .repo file in /etc/yum.repos.d
 func (c Copr) getRepoFilePath() string {
 	tmpl := template.Must(template.New("path").Parse(
 		"/etc/yum.repos.d/_copr:" + HUB + ":{{.Author}}:{{.Reponame}}.repo",
@@ -98,26 +99,28 @@ func fileExists(filename string) bool {
 	return !info.IsDir()
 }
 
+// either writes enabled=1 order enabled=0 into the repo file
 func writeEnabled(configPath string, enable int) error {
 	if !fileExists(configPath) {
 		return errors.New("File does not exist!")
 	}
-	if read, err := os.ReadFile(configPath); err != nil {
+	read, err := os.ReadFile(configPath)
+	if err != nil {
 		return err
+	}
 
+	if err := os.WriteFile(
+		configPath,
+		[]byte(
+			strings.Replace(string(read),
+				"enabled="+fmt.Sprint(1-enable), "enabled="+fmt.Sprint(enable), -1)), 0); err != nil {
+		return err
 	} else {
-		if err := os.WriteFile(
-			configPath,
-			[]byte(
-				strings.Replace(string(read),
-					"enabled="+fmt.Sprint(1-enable), "enabled="+fmt.Sprint(enable), -1)), 0); err != nil {
-			return err
-		} else {
-			return nil
-		}
+		return nil
 	}
 }
 
+// if repo is installed, it enables ist otherwise it installes it
 func (c Copr) Enable() {
 	configPath := c.getRepoFilePath()
 
@@ -141,6 +144,7 @@ func (c Copr) Enable() {
 	os.Exit(0)
 }
 
+// if repo is enabled it disables it
 func (c Copr) Disable() {
 	configPath := c.getRepoFilePath()
 
@@ -157,6 +161,7 @@ func (c Copr) Disable() {
 	}
 }
 
+// remove the repo file
 func (c Copr) Remove() {
 	configPath := c.getRepoFilePath()
 
@@ -174,6 +179,7 @@ func (c Copr) Remove() {
 	os.Exit(0)
 }
 
+// list installed coprs
 func ListCoprs(flags *pflag.FlagSet) {
 	repoConfigs, err := os.ReadDir("/etc/yum.repos.d/")