Unnecessary else block is necessary. See fca329

gkze · Dec 11, 2018 · fcc595c · fcc595c
1 parent 82afeff
commit fcc595c
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/README.md b/README.md
@@ -37,7 +37,7 @@ USAGE:
    stars [global options] command [command options] [arguments...]
 
 VERSION:
-   0.3.8
+   0.3.9
 
 COMMANDS:
      save         Save all stars

diff --git a/main.go b/main.go
@@ -21,7 +21,7 @@ func main() {
 	cmdline := cli.NewApp()
 	cmdline.Name = "stars"
 	cmdline.Usage = "Command-line interface to YOUR GitHub stars"
-	cmdline.Version = "0.3.8"
+	cmdline.Version = "0.3.9"
 	cmdline.Commands = []cli.Command{
 		{
 			Name:  "save",

diff --git a/starmanager/starmanager.go b/starmanager/starmanager.go
@@ -280,10 +280,10 @@ func (s *StarManager) GetProjects(count int, language, topic string, random bool
 		rand.Shuffle(len(stars), func(i, j int) {
 			stars[i], stars[j] = stars[j], stars[i]
 		})
+	} else {
+		sort.Slice(stars, func(i, j int) bool { return stars[i].Stargazers > stars[j].Stargazers })
 	}
 
-	sort.Slice(stars, func(i, j int) bool { return stars[i].Stargazers > stars[j].Stargazers })
-
 	if len(stars) > 0 {
 		if len(stars) > count {
 			return stars[0:count], nil