feat(changelog): include PR links in summary bullets too#13165
Conversation
|
Hey! Your PR title Please update it to start with one of:
Where See CONTRIBUTING.md for details. |
|
The following comment was made by an LLM, it may be inaccurate: |
|
Thanks for your contribution! This PR doesn't have a linked issue. All PRs must reference an existing issue. Please:
See CONTRIBUTING.md for details. |
| @@ -178,8 +183,9 @@ export async function generateChangelog(commits: Commit[], opencode: Awaited<Ret | |||
| for (let i = 0; i < commits.length; i++) { | |||
| const commit = commits[i]! | |||
| const section = getSection(commit.areas) | |||
| const prs = commit.prs.length > 0 ? ` (${commit.prs.map((n) => `#${n}`).join(", ")})` : "" | |||
There was a problem hiding this comment.
how about rendering links?
const prs = commit.prs.length > 0 ? ` (${commit.prs.map((n) => `[#${n}](https://github.com/anomalyco/opencode/pull/${n})`).join(", ")})` : ""
|
Closing this pull request because it has had no updates for more than 60 days. If you plan to continue working on it, feel free to reopen or open a new PR. |
What does this PR do?
I like the release changelog summary a lot but still, although the individual contributor list below contains the PRs, it feels more natural to also be able to directly go to the origin PRs from there.
How did you verify your code works?
I didn't really, because I'm not sure how to test something part of the release process, sorry :|
Fixes #13197