Skip to content

Commit 200e1da

Browse files
authored
Merge pull request #4 from duck8823/fix/stop_early
Fix: early return when pr merged to not default branch
2 parents fb6f56b + beebe15 commit 200e1da

1 file changed

Lines changed: 9 additions & 0 deletions

File tree

ghrn/ghrn.go

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,10 @@ func BuildReleaseNotes(ctx context.Context, w io.Writer, conf Config) error {
5757
State: "closed",
5858
}
5959

60+
repo, _, err := cl.Repositories.Get(ctx, conf.Org, conf.Repo)
61+
if err != nil {
62+
return fmt.Errorf("get repository: %+v", err)
63+
}
6064
var commitsNotMerged []string = nil
6165
// Iterate over all PRs
6266
for {
@@ -80,6 +84,11 @@ func BuildReleaseNotes(ctx context.Context, w io.Writer, conf Config) error {
8084
}
8185

8286
if conf.SinceLatestRelease {
87+
if pr.GetBase().GetRef() != repo.GetDefaultBranch() {
88+
// Skip when PR base branch isn't a default branch
89+
continue
90+
}
91+
8392
if commitsNotMerged == nil {
8493
commitsNotMerged, err = newCommits(ctx, cl, conf.Org, conf.Repo)
8594
if err != nil {

0 commit comments

Comments
 (0)