Skip to content

Validate that projects use Changesets CLI v3 and direct Changesets CLI v2 users to changesets/action@v1#699

Merged
Andarist merged 4 commits into
mainfrom
alidate-installed-cli
Jul 23, 2026
Merged

Validate that projects use Changesets CLI v3 and direct Changesets CLI v2 users to changesets/action@v1#699
Andarist merged 4 commits into
mainfrom
alidate-installed-cli

Conversation

@Andarist

Copy link
Copy Markdown
Member

No description provided.

@Andarist
Andarist requested review from beeequeue and bluwy July 22, 2026 09:30
@changeset-bot

changeset-bot Bot commented Jul 22, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 09fa258

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

This PR includes changesets to release 1 package
Name Type
@changesets/action 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

Comment thread src/utils.ts
Comment thread src/version/index.ts Outdated

// If the user needs to change the cwd, set `working-directory` in the step instead
const cwd = process.cwd();
await validateChangesetsCliVersion(cwd);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should we run this at the start of every main instead? Or directly call the function in the top-level try-catch (that calls main)?

That way our pre-checks are in a consistent spot.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

That way our pre-checks are in a consistent spot.

ye, good suggestion - I changed it accordingly

@Andarist
Andarist requested review from beeequeue and bluwy July 23, 2026 09:25
@Andarist
Andarist added this pull request to the merge queue Jul 23, 2026
Merged via the queue into main with commit 5b307d3 Jul 23, 2026
7 checks passed
@Andarist
Andarist deleted the alidate-installed-cli branch July 23, 2026 09:36
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