fix: make it so release.sh script doesn't output duplicate change PRs (#2735)
This commit is contained in:
parent
3d1ad4c64a
commit
8c09efff4c
|
|
@ -61,6 +61,8 @@ func printPullRequests() {
|
||||||
fmt.Println(fmt.Sprintf("Collecting pull request that were merged since the last release: %s (%s)", *releases[0].TagName, lastReleaseTime))
|
fmt.Println(fmt.Sprintf("Collecting pull request that were merged since the last release: %s (%s)", *releases[0].TagName, lastReleaseTime))
|
||||||
|
|
||||||
listSize := 1
|
listSize := 1
|
||||||
|
seen := map[int]bool{}
|
||||||
|
|
||||||
for page := 0; listSize > 0; page++ {
|
for page := 0; listSize > 0; page++ {
|
||||||
pullRequests, _, _ := client.PullRequests.List(context.Background(), org, repo, &github.PullRequestListOptions{
|
pullRequests, _, _ := client.PullRequests.List(context.Background(), org, repo, &github.PullRequestListOptions{
|
||||||
State: "closed",
|
State: "closed",
|
||||||
|
|
@ -75,8 +77,9 @@ func printPullRequests() {
|
||||||
for idx := range pullRequests {
|
for idx := range pullRequests {
|
||||||
pr := pullRequests[idx]
|
pr := pullRequests[idx]
|
||||||
if pr.MergedAt != nil {
|
if pr.MergedAt != nil {
|
||||||
if pr.GetMergedAt().After(lastReleaseTime.Time) {
|
if _, ok := seen[*pr.Number]; !ok && pr.GetMergedAt().After(lastReleaseTime.Time) {
|
||||||
fmt.Printf("* %s [#%d](https://github.com/%s/%s/pull/%d)\n", pr.GetTitle(), *pr.Number, org, repo, *pr.Number)
|
fmt.Printf("* %s [#%d](https://github.com/%s/%s/pull/%d)\n", pr.GetTitle(), *pr.Number, org, repo, *pr.Number)
|
||||||
|
seen[*pr.Number] = true
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue