Improve tag message #8

Add some cleanup

app/app.go

@@ -9,8 +9,6 @@ import (
 )
 
 type App interface {
-	GetZen(ctx context.Context) (string, error)
-
 	HandleWebhook(ctx context.Context, pr *github.PullRequestEvent) error
 }
 

app/create_tag.go

@@ -4,15 +4,21 @@ import (
 	"context"
 	"fmt"
 	"log"
+	"strings"
 
 	"github.com/google/go-github/v35/github"
 )
 
-func createTag(ctx context.Context, client *github.Client, repoOwner, repoName, commit, version, repoURL string, prNumber int) error {
+func createTag(ctx context.Context, client *github.Client, repoOwner, repoName, commit, version string, prNumber int) error {
+	message, err := getTagMessage(ctx, client, repoOwner, repoName, prNumber)
+	if err != nil {
+		return handleError(err)
+	}
+
 	tag := &github.Tag{
 		Tag:     &version,
 		SHA:     &commit,
-		Message: github.String(""),
+		Message: github.String(message),
 		Object: &github.GitObject{
 			Type: github.String("commit"),
 			SHA:  &commit,
@@ -33,8 +39,23 @@ func createTag(ctx context.Context, client *github.Client, repoOwner, repoName,
 		return handleError(err)
 	}
 
-	tagURL := fmt.Sprintf("%s/releases/tag/%s", repoURL, version)
-	commentBody := fmt.Sprintf("[%s](%s) created on %s", version, tagURL, commit)
+	commentBody := fmt.Sprintf("[%s](../releases/tag/%s) created on %s", version, version, commit)
 	err = createComment(ctx, client, prNumber, repoOwner, repoName, commentBody)
 	return err
 }
+
+func getTagMessage(ctx context.Context, client *github.Client, repoOwner, repoName string, prNumber int) (string, error) {
+	var message strings.Builder
+	commits, _, err := client.PullRequests.ListCommits(ctx, repoOwner, repoName, prNumber, &github.ListOptions{})
+	if err != nil {
+		return "", err
+	}
+	message.WriteString("release_notes:\n")
+	for _, commit := range commits {
+		if cm := commit.Commit.GetMessage(); cm != "" {
+			message.WriteString(fmt.Sprintf("  - [%s] %s\n", commit.GetSHA()[:9], cm))
+		}
+	}
+
+	return message.String(), nil
+}

app/handle_closed.go

@@ -49,7 +49,7 @@ func handleClosed(ctx context.Context, client *github.Client, prEvent *github.Pu
 		return err
 	}
 
-	err = createTag(ctx, client, repoOwner, repoName, commitSHA, versionNum, repo.GetHTMLURL(), pr.GetNumber())
+	err = createTag(ctx, client, repoOwner, repoName, commitSHA, versionNum, pr.GetNumber())
 	if err != nil {
 		commentBody := fmt.Sprintf("Failed to make tag: `%s`", err.Error())
 		createComment(ctx, client, pr.GetNumber(), repoOwner, repoName, commentBody)

app/handle_labeled.go

@@ -65,7 +65,7 @@ func handleLabeled(ctx context.Context, client *github.Client, prEvent *github.P
 
 	commitSHA := head.GetSHA()
 	prNum := pr.GetNumber()
-	err = createTag(ctx, client, repoOwner, repoName, commitSHA, versionNum, repo.GetHTMLURL(), prNum)
+	err = createTag(ctx, client, repoOwner, repoName, commitSHA, versionNum, prNum)
 	if err != nil {
 		return err
 	}

pkg/version/version_test.go

@@ -43,7 +43,6 @@ func TestPatchVersion(t *testing.T) {
 }
 
 func TestPreVersion(t *testing.T) {
-
 	nextVersion, err := FindNextVersion(Prerelease, v)
 	if err != nil {
 		t.Error(err)