Skip to content

fix: Remove snapshot carry on auto-update branches#786

Merged
ninjasimon merged 2 commits into
mainfrom
fix/workflow-no-snapshot
Jun 9, 2026
Merged

fix: Remove snapshot carry on auto-update branches#786
ninjasimon merged 2 commits into
mainfrom
fix/workflow-no-snapshot

Conversation

@ninjasimon

@ninjasimon ninjasimon commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

The auto-update workflow's "Keep changesets and snapshot" step copied .openapi-snapshot.yaml from the auto-update branch back into the workspace before running oasdiff. When that branch already held a snapshot updated by an earlier run, the spec diff compared current api-definitions against itself and missed every real change that happened after the snapshot was last advanced.

This PR removes the snapshot carry. Each workflow run now diffs current api-definitions against main's .openapi-snapshot.yaml, which only advances when a release ships.

@ninjasimon ninjasimon self-assigned this Jun 8, 2026
@ninjasimon ninjasimon requested a review from a team as a code owner June 8, 2026 20:38
@changeset-bot

changeset-bot Bot commented Jun 8, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: fd5f507

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@rebilly/client-php Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@ninjasimon ninjasimon merged commit d7df072 into main Jun 9, 2026
7 checks passed
@ninjasimon ninjasimon deleted the fix/workflow-no-snapshot branch June 9, 2026 13:02
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.

3 participants