Merge pull request #9 from moltam/fix/empty-tag-list

unknwon · unknwon · commit 731b9be71beb · 2016-03-23T15:36:23.000-04:00
Fixed returning non empty tag list when there are no tags.
diff --git a/repo_tag.go b/repo_tag.go
@@ -103,7 +103,8 @@ func (repo *Repository) GetTags() ([]string, error) {
 		return nil, err
 	}
 
-	tags := strings.Split(strings.TrimSpace(stdout), "\n")
+	tags := strings.Split(stdout, "\n")
+	tags = tags[:len(tags)-1]
 
 	if version.Compare(gitVersion, "2.0.0", "<") {
 		version.Sort(tags)

Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,8 @@ func (repo *Repository) GetTags() ([]string, error) {`
`103`	`103`	`return nil, err`
`104`	`104`	`}`
`105`	`105`
`106`		`- tags := strings.Split(strings.TrimSpace(stdout), "\n")`
	`106`	`+ tags := strings.Split(stdout, "\n")`
	`107`	`+ tags = tags[:len(tags)-1]`
`107`	`108`
`108`	`109`	`if version.Compare(gitVersion, "2.0.0", "<") {`
`109`	`110`	`version.Sort(tags)`