Skip to content

ci: baseline semver-checks against origin/main instead of crates.io#424

Merged
rcoh merged 1 commit into
mainfrom
ci/semver-baseline-main
May 18, 2026
Merged

ci: baseline semver-checks against origin/main instead of crates.io#424
rcoh merged 1 commit into
mainfrom
ci/semver-baseline-main

Conversation

@rcoh
Copy link
Copy Markdown
Contributor

@rcoh rcoh commented May 18, 2026

Switch the Semver checks CI job to use --baseline-rev origin/main instead of the default crates.io baseline.

Why

By default, cargo-semver-checks compares the PR against the latest crates.io release. That's good for "must I bump major before publishing?" but noisy for per-PR review:

  • Every PR sees every accumulated breaking change since the last release, drowning out what this PR introduced.
  • Stacked PRs relitigate the same advisory warnings.

--baseline-rev origin/main makes the per-PR check report only what the PR itself introduces. The "must I bump major?" check is already separately handled at release time by release-plz's own semver_check = true (per CONTRIBUTING.md), so we don't lose any release safety.

Changes

  • actions/checkout: fetch-depth: 0 (need full history for the baseline rev).
  • cargo semver-checks: add --baseline-rev origin/main.

Reference: https://github.com/obi1kenobi/cargo-semver-checks#does-the-crate-im-checking-have-to-be-published-on-cratesio

Expected effect on this PR

This PR itself touches only .github/workflows/ci.yml, so the new semver-checks job has nothing to compare and should pass.

@rcoh rcoh enabled auto-merge May 18, 2026 18:05
@rcoh rcoh added this pull request to the merge queue May 18, 2026
Merged via the queue into main with commit 53eafd4 May 18, 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