Skip to content

fix(ci): update changesets/action inputs for v2 - #143

Merged
iivvaannxx merged 2 commits into
mainfrom
fix/changesets-action-v2-inputs
Sep 18, 2026
Merged

iivvaannxx merged 2 commits into
mainfrom
fix/changesets-action-v2-inputs

Conversation

@iivvaannxx

@iivvaannxx iivvaannxx commented Sep 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • The renovate PR that bumped changesets/action to v2 (chore(renovate): Update changesets/action action to v2 #136) didn't account for the action's breaking changes, breaking the Publish workflow on main right after the build step succeeds.
  • changesets/action v2 renamed inputs versionversion-script, publishpublish-script, commitcommit-message, titlepr-title, and renamed the publishedPackages output to published-packages. Updated .github/workflows/publish.yml for both (the output rename was missed by the equivalent fix in RENOVATE: Update changesets/action action to v2 aio-commerce-sdk#622 — confirmed against the action's action.yml at the pinned SHA).
  • Bumped .changeset/config.json's $schema to @changesets/config@4.0.0 to match the @changesets/cli v3 bump.
  • Also checked the @changesets/cli v3 and @changesets/changelog-github v1 major-version changes (private-package versioning defaults, prettierformat config, changeset taggit-tag rename, Node engine bump) — none apply to this repo's config or scripts.

Test plan

  • Confirmed the failing run (https://github.com/adobe/aio-lib-telemetry/actions/runs/35261674576) errors on changesets/action with "The following inputs have been renamed"
  • Confirmed via changesets/action's action.yml that publishedPackages output was renamed to published-packages
  • Merge and confirm the next Publish run on main completes past the Create Release Pull Request step and, on the next actual publish, the Slack announcement payload is non-empty

changesets/action v2 renamed its inputs (version->version-script,
publish->publish-script, commit->commit-message, title->pr-title),
which broke the Publish workflow on main after the renovate bump.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@changeset-bot

changeset-bot Bot commented Sep 18, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: a29da68

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@iivvaannxx
iivvaannxx requested a review from a team as a code owner September 18, 2026 09:54
The Slack announcement step still referenced the pre-v2 output name
publishedPackages instead of published-packages, which would have
produced an empty payload on the next successful publish. Also bump
the .changeset/config.json $schema to match @changesets/config@4.0.0.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@iivvaannxx
iivvaannxx merged commit 7701387 into main Sep 18, 2026
5 checks passed
@iivvaannxx
iivvaannxx deleted the fix/changesets-action-v2-inputs branch September 18, 2026 10:07
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