add --ignore-authors

so that dependabot doesn't take over the release notes Signed-off-by: Bohan Chen <[email protected]> Co-authored-by: Esteban Foronda <[email protected]>
clarafu · Apr 7, 2021 · 9bd067c · 9bd067c
1 parent 838d15b
commit 9bd067c
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 3 deletions.
diff --git a/cmd/generate.go b/cmd/generate.go
@@ -4,6 +4,7 @@ import (
 	"fmt"
 	"os"
 	"regexp"
+	"strings"
 
 	"github.com/clarafu/release-me/generate"
 	"github.com/clarafu/release-me/github"
@@ -23,6 +24,7 @@ func init() {
 	generateCmd.Flags().String("github-branch", "master", "the branch name of the github repository to pull the pull requests from")
 	generateCmd.Flags().String("last-commit-SHA", "", "will generate a release note using all prs merged up to this commit SHA. If empty, will generate release note until latest commit.")
 	generateCmd.Flags().String("release-version", "", "the version that the release note will be generated for")
+	generateCmd.Flags().String("ignore-authors", "", "comma separated list of github handles, any PRs authored by these handles will be ignored.")
 	generateCmd.Flags().String("ignore-release-regex", "", "a regular expression indicating releases to ignore when determining the previous release")
 	generateCmd.MarkFlagRequired("release-version")
 }
@@ -72,10 +74,13 @@ func generateReleaseNote(cmd *cobra.Command, args []string) {
 
 	lastCommitSHA, _ := cmd.Flags().GetString("last-commit-SHA")
 
+	ignoreAuthorsStr, _ := cmd.Flags().GetString("ignore-authors")
+	ignoreAuthors := strings.Split(ignoreAuthorsStr, ",")
+
 	// Fetch all pull requests that are associated to a commit after the starting
 	// commit SHA. If the pull request is already used for a patch release, it is
 	// not included.
-	pullRequests, err := client.FetchPullRequestsAfterCommit(githubOwner, githubRepo, githubBranch, startingCommitSHA, lastCommitSHA)
+	pullRequests, err := client.FetchPullRequestsAfterCommit(githubOwner, githubRepo, githubBranch, startingCommitSHA, lastCommitSHA, ignoreAuthors)
 	if err != nil {
 		failf("failed to fetch pull requests: %s", err)
 	}

diff --git a/github/github.go b/github/github.go
@@ -138,7 +138,7 @@ func (g GitHub) FetchLatestReleaseCommitFromBranch(owner, repo, branch, versionT
 	return lastCommit, nil
 }
 
-func (g GitHub) FetchPullRequestsAfterCommit(owner, repo, branch, startingCommitSHA, lastCommitSHA string) ([]PullRequest, error) {
+func (g GitHub) FetchPullRequestsAfterCommit(owner, repo, branch, startingCommitSHA, lastCommitSHA string, ignoreAuthors []string) ([]PullRequest, error) {
 	var pullRequestsQuery struct {
 		Repository struct {
 			Ref struct {
@@ -186,7 +186,12 @@ func (g GitHub) FetchPullRequestsAfterCommit(owner, repo, branch, startingCommit
 
 	var appendCommits bool
 	pullRequests := []PullRequest{}
-	seen := map[string]bool{}
+	seen := make(map[string]bool)
+
+	filteredAuthors := make(map[string]struct{})
+	for _, username := range ignoreAuthors {
+		filteredAuthors[username] = struct{}{}
+	}
 
 	for {
 		err := g.client.Query(context.Background(), &pullRequestsQuery, pullRequestsVariables)
@@ -212,6 +217,10 @@ func (g GitHub) FetchPullRequestsAfterCommit(owner, repo, branch, startingCommit
 					continue
 				}
 
+				if _, found := filteredAuthors[pr.Author.Login]; found {
+					continue
+				}
+
 				seen[pr.ID] = true
 
 				if appendCommits {