FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

ci: use the input names changesets/action v2 expects by alexander-akait · Pull Request #4838 · webpack/webpack-cli · GitHub

ci: use the input names changesets/action v2 expects - #4838

Merged
alexander-akait merged 1 commit into
mainfrom
ci/fix-changesets-action-inputs
Aug 28, 2026
Merged

alexander-akait merged 1 commit into
mainfrom
ci/fix-changesets-action-inputs

Conversation

Copy link
Copy Markdown
Member

Summary

The Release workflow has failed on every push to main since #4831 bumped changesets/action to v2, which renamed every input the workflow passes and errors out rather than ignoring the old ones: version → version-script, publish → publish-script, commit → commit-message, title → pr-title. Run 121 (11 Aug) was the last success.

Worth noting for review: the action's own error message misspells one of them as commit-mesage. action.yml at the pinned commit spells it commit-message, and taking the error at its word would silently fall back to the default Version Packages commit message. The one output this workflow consumes, steps.changesets.outputs.published, is unchanged in v2.

What kind of change does this PR introduce?

ci

Did you add tests for your changes?

No — a workflow input rename that no test can exercise. Each name was checked against action.yml at the pinned commit 198f833, and the file was verified to parse with the four inputs set as intended. It proves itself on the next push to main, since the workflow has no manual trigger.

Does this PR introduce a breaking change?

No.

If relevant, what needs to be documented once your changes are merged or what have you already documented?

n/a — though note that seven releases have been skipped while this was broken, so the first successful run will open a version PR covering everything accumulated since 11 Aug.

Use of AI

Claude Code was used to read the failing job log, trace the breakage back to the run that introduced it, check the input names against the pinned action's action.yml (catching the typo in the error message), and apply the rename. The change was reviewed before committing.


Generated by Claude Code

v2 renamed every input this workflow passes, so the release job has failed
since the action was bumped in #4831: `version` is `version-script`,
`publish` is `publish-script`, `commit` is `commit-message` and `title` is
`pr-title`.

Note the action's own error message misspells one of them as
`commit-mesage`; `action.yml` at the pinned commit spells it
`commit-message`, and taking the error at its word would silently restore
the default "Version Packages" commit message.

changeset-bot Bot commented Aug 28, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 5f60c23

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

github-actions Bot commented Aug 28, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

This PR is packaged and the instant preview is available (1f6593a).

Install it locally:

  • npm
npm i -D create-webpack-app@https://pkg.pr.new/create-webpack-app@1f6593a webpack-cli@https://pkg.pr.new/webpack-cli@1f6593a
  • yarn
yarn add -D create-webpack-app@https://pkg.pr.new/create-webpack-app@1f6593a webpack-cli@https://pkg.pr.new/webpack-cli@1f6593a
  • pnpm
pnpm add -D create-webpack-app@https://pkg.pr.new/create-webpack-app@1f6593a webpack-cli@https://pkg.pr.new/webpack-cli@1f6593a

codecov Bot commented Aug 28, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.80%. Comparing base (99cfc4f) to head (5f60c23).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4838      +/-   ##
==========================================
- Coverage   94.95%   94.80%   -0.15%     
==========================================
  Files          14       14              
  Lines        5406     5406              
  Branches      802      799       -3     
==========================================
- Hits         5133     5125       -8     
- Misses        272      280       +8     
  Partials        1        1              

see 3 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 99cfc4f...5f60c23. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

alexander-akait merged commit 1f6593a into main Aug 28, 2026
40 of 41 checks passed
alexander-akait deleted the ci/fix-changesets-action-inputs branch August 28, 2026 10:51
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

1 participant


Back | FazBrowse Home | New Git URL