Skip to content
This repository has been archived by the owner on Jan 10, 2025. It is now read-only.

CI: Add git-cliff step during publish #7203

Merged
merged 5 commits into from
Aug 28, 2024
Merged

Conversation

joncinque
Copy link
Contributor

Problem

The publish step is great, but there's still some manual work to add the changelog and correct title for the created release.

Summary of changes

Use git-cliff to generate the release notes. This comes with a very simple cliff.toml which will do minimal processing, since we don't follow conventional commits in the repo.

Here's a test run for the given input, which filters out all dependabot bumps, which are typically less interesting.

$ git-cliff -c ci/cliff.toml "pod-v0.3.0"..master --include-path "libraries/pod/**" --github-repo "solana-labs/solana-program-library"

Output:

## What's new

- Publish pod v0.3.2 by @github-actions[bot]
- [token-2022] Upgrade to `zk-sdk` (#7148) by @samkim-crypto
- Implement `Default` for `PodOption` (#7083) by @joncinque
- Bump to 0.3.1 (#7075) by @febo
- Improve `PodOption` type (#7076) by @febo
- Add `PodU128` type (#7012) by @febo
- Add `PodOption` type (#6886) by @febo
- Use `bytemuck_derive` explicitly (#6928) by @joncinque

Note: we (I?) missed publishing pod-v0.3.1, which is why this picks up the previous bump too.

#### Problem

The publish step is great, but there's still some manual work to add the
changelog and correct title for the created release.

#### Summary of changes

Use git-cliff to generate the release notes. This comes with a very
simple cliff.toml which will do minimal processing, since we don't
follow conventional commits in the repo.

Here's a test run for the given input:

```
$ git-cliff -c ci/cliff.toml "pod-v0.3.0"..master --include-path "libraries/pod/**" --github-repo "solana-labs/solana-program-library"
```

Output:

```
## What's new

- Publish pod v0.3.2 by @github-actions[bot]
- [token-2022] Upgrade to `zk-sdk` (solana-labs#7148) by @samkim-crypto
- Implement `Default` for `PodOption` (solana-labs#7083) by @joncinque
- Bump to 0.3.1 (solana-labs#7075) by @febo
- Improve `PodOption` type (solana-labs#7076) by @febo
- Add `PodU128` type (solana-labs#7012) by @febo
- Add `PodOption` type (solana-labs#6886) by @febo
- Use `bytemuck_derive` explicitly (solana-labs#6928) by @joncinque
```
@joncinque joncinque requested review from lorisleiva and yihau August 27, 2024 21:10
yihau
yihau previously approved these changes Aug 28, 2024
Copy link
Contributor

@yihau yihau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it looks great to have this one! btw, looks like there is a conflict in this PR haha. will approve this one very quickly again when we solve it!

@mergify mergify bot dismissed yihau’s stale review August 28, 2024 09:48

Pull request has been modified.

@joncinque joncinque merged commit beca5b9 into solana-labs:master Aug 28, 2024
6 checks passed
@joncinque joncinque deleted the gitcliff branch August 28, 2024 09:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants