Skip to content

fix(ci): pin changesets/action to cwd-fix commit#12506

Merged
mikeldking merged 1 commit intomainfrom
fix-ci-js
Apr 2, 2026
Merged

fix(ci): pin changesets/action to cwd-fix commit#12506
mikeldking merged 1 commit intomainfrom
fix-ci-js

Conversation

@mikeldking
Copy link
Copy Markdown
Collaborator

Summary

  • Pins changesets/action to a post-fix commit that properly passes cwd to readChangesetState() and runVersion()
  • Fixes "There is no .changeset directory in this project" error in CI (Error: There is no .changeset directory in this project changesets/action#501)
  • v1.7.0 has a bug where the cwd input is ignored during the initial .changeset directory check, so it always looks at the repo root instead of ./js/

Test plan

  • Merge a changeset and verify the publish workflow no longer errors

v1.7.0 doesn't pass the `cwd` parameter to `readChangesetState()`,
causing "There is no .changeset directory" errors in monorepos.
See changesets/action#501.
@github-project-automation github-project-automation bot moved this to 📘 Todo in phoenix Apr 2, 2026
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Apr 2, 2026
@mintlify
Copy link
Copy Markdown
Contributor

mintlify bot commented Apr 2, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
arize-phoenix 🟢 Ready View Preview Apr 2, 2026, 4:19 AM

@mikeldking mikeldking merged commit d197512 into main Apr 2, 2026
42 checks passed
@mikeldking mikeldking deleted the fix-ci-js branch April 2, 2026 04:20
@github-project-automation github-project-automation bot moved this from 📘 Todo to ✅ Done in phoenix Apr 2, 2026
# Using PAT instead of GITHUB_TOKEN so that PRs created by this action
# can trigger subsequent workflows (like publish) when merged
uses: changesets/action@6a0a831ff30acef54f2c6aa1cbbc1096b066edaf # v1.7.0
uses: changesets/action@2ae596f3dd74aaee4f346b31fda33a58528d3d40 # v1.7.0+cwd-fix (changesets/action#501)
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.

The commit hash 2ae596f3dd74aaee4f346b31fda33a58528d3d40 does not appear to exist on the main branch of changesets/action, which is a supply chain risk — a commit not on main could be force-pushed away or garbage-collected.

A few things to verify before merging:

  1. changesets/action#501 is an issue, not a PR. The reference is misleading. The actual fix PRs that landed on main on March 31, 2026 are:

  2. Consider pinning to the merge commit of PR [ui] display evaluation window on timeseries graphs #536 (81b3f61ebffcb868f73e4c0b2682517149c834a2), which is the more complete fix and is confirmed on main.

Suggested change
uses: changesets/action@2ae596f3dd74aaee4f346b31fda33a58528d3d40 # v1.7.0+cwd-fix (changesets/action#501)
uses: changesets/action@81b3f61ebffcb868f73e4c0b2682517149c834a2 # v1.7.0+cwd-fix (changesets/action#536)

KaloyanBorisov pushed a commit to KaloyanBorisov/phoenix that referenced this pull request Apr 2, 2026
v1.7.0 doesn't pass the `cwd` parameter to `readChangesetState()`,
causing "There is no .changeset directory" errors in monorepos.
See changesets/action#501.
KaloyanBorisov pushed a commit to KaloyanBorisov/phoenix that referenced this pull request Apr 12, 2026
v1.7.0 doesn't pass the `cwd` parameter to `readChangesetState()`,
causing "There is no .changeset directory" errors in monorepos.
See changesets/action#501.
github-actions bot pushed a commit to KaloyanBorisov/phoenix that referenced this pull request Apr 14, 2026
v1.7.0 doesn't pass the `cwd` parameter to `readChangesetState()`,
causing "There is no .changeset directory" errors in monorepos.
See changesets/action#501.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

1 participant