feat: Add Commits To Unreleased Even When No Tags

When there are commits but no tags, then insert those commits under
the unreleased section of the change log output. This requires get
an updated CHANGELOG.md.tpl file. Includig the template footer to
account for the unreleased line there that used .Versions.

resolves git-chglog#76

BREAKING CHANGE:

* No longer throw the error "no tags" when:
   * No tag query entered
   * There are commits to tag
* Requires an update to the changelog template

chglog.go

@@ -162,7 +162,7 @@ func (gen *Generator) Generate(w io.Writer, query string) error {
 		return err
 	}
 
-	if len(versions) == 0 {
+	if len(versions) == 0 && query != "" {
 		return fmt.Errorf("commits corresponding to \"%s\" was not found", query)
 	}
 
@@ -284,7 +284,8 @@ func (gen *Generator) getTags(query string) ([]*Tag, string, error) {
 		}, tags...)
 	}
 
-	if len(tags) == 0 {
+	// Error only when you explicitly set a query
+	if len(tags) == 0 && query != "" {
 		return nil, "", errors.New("git-tag does not exist")
 	}
 

chglog_test.go

@@ -70,7 +70,7 @@ func cleanup() {
 
 func TestGeneratorNotFoundTags(t *testing.T) {
 	assert := assert.New(t)
-	testName := "not_found"
+	testName := "no_tags"
 
 	setup(testName, func(commit commitFunc, _ tagFunc, _ gitcmd.Client) {
 		commit("2018-01-01 00:00:00", "feat(*): New feature", "")
@@ -91,9 +91,8 @@ func TestGeneratorNotFoundTags(t *testing.T) {
 	err := gen.Generate(buf, "")
 	expected := strings.TrimSpace(buf.String())
 
-	assert.Error(err)
-	assert.Contains(err.Error(), "git-tag does not exist")
-	assert.Equal("", expected)
+	assert.Nil(err)
+	assert.Contains(expected, "<a name=\"unreleased\"></a>\n## [Unreleased]")
 }
 
 func TestGeneratorNotFoundCommits(t *testing.T) {
@@ -629,3 +628,50 @@ func TestGeneratorWithSprig(t *testing.T) {
 [2.0.0]: https://github.com/git-chglog/git-chglog/compare/1.0.0...2.0.0`, expected)
 
 }
+
+func TestNewRepoWithCommitsButNoTags(t *testing.T) {
+	asrt := assert.New(t)
+	testName := "commits_and_no_tags"
+
+	setup(testName, func(commit commitFunc, tag tagFunc, _ gitcmd.Client) {
+		commit("2023-06-25 00:00:00", "feat: Second Feature", "")
+		commit("2023-06-24 00:00:00", "feat: First Feature", "")
+	})
+
+	gen := NewGenerator(NewLogger(os.Stdout, os.Stderr, false, true),
+		&Config{
+			Bin:        "git",
+			WorkingDir: filepath.Join(testRepoRoot, testName),
+			Template:   filepath.Join(cwd, "testdata", testName+".md"),
+			Info: &Info{
+				Title:         "CHANGELOG Example",
+				RepositoryURL: "https://github.com/git-chglog/git-chglog",
+			},
+			Options: &Options{
+				CommitGroupBy: "Type",
+				CommitGroupTitleMaps: map[string]string{
+					"feat": "Features",
+				},
+				HeaderPattern: "^(\\w*)\\:\\s(.*)$",
+				HeaderPatternMaps: []string{
+					"Type",
+					"Subject",
+				}},
+		})
+
+	buf := &bytes.Buffer{}
+	err := gen.Generate(buf, "")
+	actual := strings.TrimSpace(buf.String())
+
+	asrt.Nil(err)
+	asrt.Contains(actual, `<a name="unreleased"></a>
+## [Unreleased]
+
+### Features
+- First Feature
+- Second Feature
+
+
+[Unreleased]: https://github.com/git-chglog/git-chglog/compare/`)
+
+}

cmd/git-chglog/kac_template_builder.go

@@ -57,7 +57,7 @@ func (t *kacTemplateBuilderImpl) unreleased(style, format string) string {
 		title = fmt.Sprintf("[%s]", title)
 	}
 
-	return fmt.Sprintf(`{{ if .Versions -}}
+	return fmt.Sprintf(`{{ if .Unreleased -}}
 %s## %s
 
 {{ if .Unreleased.CommitGroups -}}
@@ -156,24 +156,34 @@ func (*kacTemplateBuilderImpl) footer(style string) string {
 	case styleGitHub, styleGitLab:
 		return `
 
-{{- if .Versions }}
-[Unreleased]: {{ .Info.RepositoryURL }}/compare/{{ $latest := index .Versions 0 }}{{ $latest.Tag.Name }}...HEAD
+{{- if .Unreleased -}}
+{{- $latest := index .Unreleased.Commits 0 -}}
+{{- if .Unreleased.CommitGroups -}}
+{{- $group := index .Unreleased.CommitGroups 0 -}}
+{{- $latest = index $group.Commits 0 -}}
+{{- end }}
+[Unreleased]: {{ .Info.RepositoryURL }}/compare/{{ $latest.Hash }}...HEAD
+{{ end -}}
 {{ range .Versions -}}
 {{ if .Tag.Previous -}}
 [{{ .Tag.Name }}]: {{ $.Info.RepositoryURL }}/compare/{{ .Tag.Previous.Name }}...{{ .Tag.Name }}
 {{ end -}}
-{{ end -}}
 {{ end -}}`
 	case styleBitbucket:
 		return `
 
-{{- if .Versions }}
-[Unreleased]: {{ .Info.RepositoryURL }}/compare/HEAD..{{ $latest := index .Versions 0 }}{{ $latest.Tag.Name }}
+{{- if .Unreleased -}}
+{{- $latest := index .Unreleased.Commits 0 -}}
+{{- if .Unreleased.CommitGroups -}}
+{{- $group := index .Unreleased.CommitGroups 0 -}}
+{{- $latest = index $group.Commits 0 -}}
+{{- end }}
+[Unreleased]: {{ .Info.RepositoryURL }}/compare/{{ $latest.Hash }}...HEAD
+{{ end -}}
 {{ range .Versions -}}
 {{ if .Tag.Previous -}}
 [{{ .Tag.Name }}]: {{ $.Info.RepositoryURL }}/compare/{{ .Tag.Name }}..{{ .Tag.Previous.Name }}
 {{ end -}}
-{{ end -}}
 {{ end -}}`
 	default:
 		return ""

cmd/git-chglog/kac_template_builder_test.go

@@ -19,7 +19,7 @@ func TestKACTemplateBuilderDefault(t *testing.T) {
 	})
 
 	assert.Nil(err)
-	assert.Equal(`{{ if .Versions -}}
+	assert.Equal(`{{ if .Unreleased -}}
 <a name="unreleased"></a>
 ## [Unreleased]
 
@@ -67,13 +67,18 @@ func TestKACTemplateBuilderDefault(t *testing.T) {
 {{ end -}}
 {{ end -}}
 
-{{- if .Versions }}
-[Unreleased]: {{ .Info.RepositoryURL }}/compare/{{ $latest := index .Versions 0 }}{{ $latest.Tag.Name }}...HEAD
+{{- if .Unreleased -}}
+{{- $latest := index .Unreleased.Commits 0 -}}
+{{- if .Unreleased.CommitGroups -}}
+{{- $group := index .Unreleased.CommitGroups 0 -}}
+{{- $latest = index $group.Commits 0 -}}
+{{- end }}
+[Unreleased]: {{ .Info.RepositoryURL }}/compare/{{ $latest.Hash }}...HEAD
+{{ end -}}
 {{ range .Versions -}}
 {{ if .Tag.Previous -}}
 [{{ .Tag.Name }}]: {{ $.Info.RepositoryURL }}/compare/{{ .Tag.Previous.Name }}...{{ .Tag.Name }}
 {{ end -}}
-{{ end -}}
 {{ end -}}`, out)
 }
 
@@ -90,7 +95,7 @@ func TestKACTemplateBuilderNone(t *testing.T) {
 	})
 
 	assert.Nil(err)
-	assert.Equal(`{{ if .Versions -}}
+	assert.Equal(`{{ if .Unreleased -}}
 ## Unreleased
 
 {{ if .Unreleased.CommitGroups -}}
@@ -150,7 +155,7 @@ func TestKACTemplateBuilderSubject(t *testing.T) {
 	})
 
 	assert.Nil(err)
-	assert.Equal(`{{ if .Versions -}}
+	assert.Equal(`{{ if .Unreleased -}}
 ## Unreleased
 
 {{ if .Unreleased.CommitGroups -}}

testdata/commits_and_no_tags.md

@@ -0,0 +1,54 @@
+{{ if .Unreleased -}}
+<a name="unreleased"></a>
+## [Unreleased]
+
+{{ if .Unreleased.CommitGroups -}}
+{{ range .Unreleased.CommitGroups -}}
+### {{ .Title }}
+{{ range .Commits -}}
+- {{ if .Scope }}**{{ .Scope }}:** {{ end }}{{ .Subject }}
+{{ end }}
+{{ end -}}
+{{ end -}}
+{{ end -}}
+
+{{ range .Versions }}
+<a name="{{ .Tag.Name }}"></a>
+## {{ if .Tag.Previous }}[{{ .Tag.Name }}]{{ else }}{{ .Tag.Name }}{{ end }} - {{ datetime "2006-01-02" .Tag.Date }}
+{{ range .CommitGroups -}}
+### {{ .Title }}
+{{ range .Commits -}}
+- {{ if .Scope }}**{{ .Scope }}:** {{ end }}{{ .Subject }}
+{{ end }}
+{{ end -}}
+
+{{- if .RevertCommits -}}
+### Reverts
+{{ range .RevertCommits -}}
+- {{ .Revert.Header }}
+{{ end }}
+{{ end -}}
+
+{{- if .NoteGroups -}}
+{{ range .NoteGroups -}}
+### {{ .Title }}
+{{ range .Notes }}
+{{ .Body }}
+{{ end }}
+{{ end -}}
+{{ end -}}
+{{ end -}}
+
+{{- if .Unreleased -}}
+{{- $latest := index .Unreleased.Commits 0 -}}
+{{- if .Unreleased.CommitGroups -}}
+{{- $group := index .Unreleased.CommitGroups 0 -}}
+{{- $latest = index $group.Commits 0 -}}
+{{- end }}
+[Unreleased]: {{ .Info.RepositoryURL }}/compare/{{ $latest.Hash }}...HEAD
+{{ end -}}
+{{ range .Versions -}}
+{{ if .Tag.Previous -}}
+[{{ .Tag.Name }}]: {{ $.Info.RepositoryURL }}/compare/{{ .Tag.Previous.Name }}...{{ .Tag.Name }}
+{{ end -}}
+{{ end -}}

testdata/no_tags.md

@@ -0,0 +1,54 @@
+{{ if .Unreleased -}}
+<a name="unreleased"></a>
+## [Unreleased]
+
+{{ if .Unreleased.CommitGroups -}}
+{{ range .Unreleased.CommitGroups -}}
+### {{ .Title }}
+{{ range .Commits -}}
+- {{ if .Scope }}**{{ .Scope }}:** {{ end }}{{ .Subject }}
+{{ end }}
+{{ end -}}
+{{ end -}}
+{{ end -}}
+
+{{ range .Versions }}
+<a name="{{ .Tag.Name }}"></a>
+## {{ if .Tag.Previous }}[{{ .Tag.Name }}]{{ else }}{{ .Tag.Name }}{{ end }} - {{ datetime "2006-01-02" .Tag.Date }}
+{{ range .CommitGroups -}}
+### {{ .Title }}
+{{ range .Commits -}}
+- {{ if .Scope }}**{{ .Scope }}:** {{ end }}{{ .Subject }}
+{{ end }}
+{{ end -}}
+
+{{- if .RevertCommits -}}
+### Reverts
+{{ range .RevertCommits -}}
+- {{ .Revert.Header }}
+{{ end }}
+{{ end -}}
+
+{{- if .NoteGroups -}}
+{{ range .NoteGroups -}}
+### {{ .Title }}
+{{ range .Notes }}
+{{ .Body }}
+{{ end }}
+{{ end -}}
+{{ end -}}
+{{ end -}}
+
+{{- if .Unreleased -}}
+{{- $latest := index .Unreleased.Commits 0 -}}
+{{- if .Unreleased.CommitGroups -}}
+{{- $group = index .Unreleased.CommitGroups 0 -}}
+{{- $latest = index $group.Commits 0 -}}
+{{- end }}
+[Unreleased]: {{ .Info.RepositoryURL }}/compare/{{ $latest.Hash }}...HEAD
+{{ end -}}
+{{ range .Versions -}}
+{{ if .Tag.Previous -}}
+[{{ .Tag.Name }}]: {{ $.Info.RepositoryURL }}/compare/{{ .Tag.Previous.Name }}...{{ .Tag.Name }}
+{{ end -}}
+{{ end -}}