Skip to content

fix: Improve API spec summarization and versioning#790

Open
ninjasimon wants to merge 2 commits into
mainfrom
fix/spec-diff-workflow-improvements
Open

fix: Improve API spec summarization and versioning#790
ninjasimon wants to merge 2 commits into
mainfrom
fix/spec-diff-workflow-improvements

Conversation

@ninjasimon

Copy link
Copy Markdown
Contributor

This PR changes the update.yaml workflow to write spec-diff changeset to a fixed .changeset/spec-diff.md path with a
hardcoded patch bump so each workflow run overwrites the previous file rather than generating a new random-named one.

Also improves deduplication in the summarization script and improves wording of the generated changesets.

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

changeset-bot Bot commented Jun 11, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: de74b11

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

working-directory: ./sdk
run: |
npm ci
node ./scripts/add-changeset.js ${{ steps.oasdiff.outputs.bump }} "$(cat /tmp/changeset-body.md)"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can we remove add-changeset.js file and @changesets/write from package.json?

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