Merge pull request #31 from isacikgoz/develop

minor bugfix and some polishment on code style

2 files changed, 6 insertions, 5 deletions

diff --git a/pkg/app/config.go b/pkg/app/config.go
index 94052e6..b613532 100644
--- a/pkg/app/config.go
+++ b/pkg/app/config.go
@@ -80,10 +80,8 @@ func readConfiguration() error {
 
 // write configuration to a file
 func writeConfiguration() error {
-	if err := viper.WriteConfig(); err != nil {
-		return err
-	}
-	return nil
+	err := viper.WriteConfig()
+	return err
 }
 
 // initialize the configuration manager
diff --git a/pkg/app/files.go b/pkg/app/files.go
index b6ec662..25e5074 100644
--- a/pkg/app/files.go
+++ b/pkg/app/files.go
@@ -74,12 +74,15 @@ func seperateDirectories(directory string) (directories, gitDirectories []string
 			return nil, nil, err
 		}
 		// with this approach, we ignore submodule or sub repositoreis in a git repository
-		_, err = os.Open(dir + string(os.PathSeparator) + ".git")
+		ff, err := os.Open(dir + string(os.PathSeparator) + ".git")
 		if err != nil {
 			directories = append(directories, dir)
 		} else {
 			gitDirectories = append(gitDirectories, dir)
 		}
+		ff.Close()
+		file.Close()
+
 	}
 	return directories, gitDirectories, nil
 }