Skip to content

Commit 6898381

Browse files
committed
amend pr feedback
1 parent f8f496c commit 6898381

File tree

2 files changed

+24
-4
lines changed

2 files changed

+24
-4
lines changed

scripts/release/main.go

+4-4
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ func promoteVersionToStable(ctx context.Context, inv *serpent.Invocation, logger
9696
// Check if the version is already the latest stable release.
9797
currentStable, _, err := client.Repositories.GetLatestRelease(ctx, "coder", "coder")
9898
if err != nil {
99-
return err
99+
return xerrors.Errorf("get latest release failed: %w", err)
100100
}
101101

102102
logger = logger.With(slog.F("stable_version", currentStable.GetTagName()))
@@ -113,7 +113,7 @@ func promoteVersionToStable(ctx context.Context, inv *serpent.Invocation, logger
113113
PerPage: perPage,
114114
})
115115
if err != nil {
116-
return err
116+
return xerrors.Errorf("list releases failed: %w", err)
117117
}
118118

119119
var releaseVersions []string
@@ -164,9 +164,9 @@ func promoteVersionToStable(ctx context.Context, inv *serpent.Invocation, logger
164164
if !dryRun {
165165
_, _, err = client.Repositories.EditRelease(ctx, owner, repo, newStable.GetID(), newStable)
166166
if err != nil {
167-
return err
167+
return xerrors.Errorf("edit release failed: %w", err)
168168
}
169-
logger.Info(ctx, "selected version promoted to stable")
169+
logger.Info(ctx, "selected version promoted to stable", "url", newStable.GetHTMLURL())
170170
} else {
171171
logger.Info(ctx, "dry-run: release not updated", "uncommitted_changes", cmp.Diff(newStable, updatedNewStable))
172172
}

scripts/release/main_test.go

+20
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,26 @@ Compare: [` + "`" + `v2.10.1...v2.10.2` + "`" + `](https://github.com/coder/code
8686
## Install/upgrade
8787
8888
Refer to our docs to [install](https://coder.com/docs/v2/latest/install) or [upgrade](https://coder.com/docs/v2/latest/admin/upgrade) Coder, or use a release asset below.
89+
`,
90+
},
91+
{
92+
name: "EntireQuotedBlurbIsRemoved",
93+
body: `## Changelog
94+
95+
> [!NOTE]
96+
> This is a mainline Coder release. We advise enterprise customers without a staging environment to install our [latest stable release](https://github.com/coder/coder/releases/latest) while we refine this version. Learn more about our [Release Schedule](https://coder.com/docs/v2/latest/install/releases).
97+
> This is an extended note.
98+
> This is another extended note.
99+
100+
### Best release yet!
101+
102+
Enjoy.
103+
`,
104+
want: `## Changelog
105+
106+
### Best release yet!
107+
108+
Enjoy.
89109
`,
90110
},
91111
}

0 commit comments

Comments
 (0)