Skip to content

Rework release CI & add semver checks#409

Merged
rcoh merged 5 commits into
mainfrom
semver-checks-ci
May 15, 2026
Merged

Rework release CI & add semver checks#409
rcoh merged 5 commits into
mainfrom
semver-checks-ci

Conversation

@rcoh
Copy link
Copy Markdown
Contributor

@rcoh rcoh commented May 15, 2026

This reworks CI slightly:

  1. Releases are not automatic anymore. They MUST be triggered manually. So the flow is now: merge release PR -> cut release. The doc now covers this. This is a little bit more secure and it also makes releasing a bit cleaner.

  2. Added tags for all github actions.

Comment thread CONTRIBUTING.md
```
1. Merge your PRs to `main` using conventional commit messages (e.g. `feat:`, `fix:`, `feat!:` for breaking changes).
2. The release PR will update automatically. Review the changelog and version bumps.
3. If you need to adjust versions (e.g. force a major bump), edit `Cargo.toml` versions in the release PR before merging.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Might also be worth mentioning changelog improvements / adding code samples is encouraged?

Comment thread CONTRIBUTING.md
2. The release PR will update automatically. Review the changelog and version bumps.
3. If you need to adjust versions (e.g. force a major bump), edit `Cargo.toml` versions in the release PR before merging.
4. Merge the release PR.
5. Go to Actions → "Publish release" → Run workflow → confirm.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We did it this way to keep the token narrower?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

it also means we can decouple changing versions in cargo.toml and clicking release

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Wouldn't we get that anyway if we only released from the release-plz pr?

But anyway this is fine

@rcoh rcoh enabled auto-merge May 15, 2026 19:20
@rcoh rcoh added this pull request to the merge queue May 15, 2026
Merged via the queue into main with commit 8b78f2b May 15, 2026
20 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants